Skip to content

Fixing source mismatch with the container.#1107

Merged
dkijania merged 4 commits intoMinaProtocol:mainfrom
EmrePiconbello:patch-1
Mar 31, 2026
Merged

Fixing source mismatch with the container.#1107
dkijania merged 4 commits intoMinaProtocol:mainfrom
EmrePiconbello:patch-1

Conversation

@EmrePiconbello
Copy link
Copy Markdown
Contributor

No description provided.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 8, 2025

@EmrePiconbello is attempting to deploy a commit to the Mina Foundation's projects Team on Vercel.

A member of the Team first needs to authorize it.

@EmrePiconbello
Copy link
Copy Markdown
Contributor Author

I just go over again and looks like we are creating the private key in /data/.mina-config while whole container and logs outputs etc happens on /root/.mina-config which is on normal docker documentation with --mount "type=bind,source=$(pwd)/.mina-config,dst=/root/.mina-config" \ I am not sure which part of this docker compose should change but it should be in parity with the rest.

@dkijania
Copy link
Copy Markdown
Member

@EmrePiconbello yes, I also have this feeling that we should use /root/.mina-config/ can you also fix remaining places where /data/.mina-config/ is used?

…config

The container's home directory is /root, so .mina-config lives at
/root/.mina-config. Updates volume mounts and internal path references
across all affected docs pages.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@EmrePiconbello
Copy link
Copy Markdown
Contributor Author

@dkijania This doesn't look exactly right for me but it's been so long since I made the deployment. I remember the reasons behind this PR but leaving rest of the edits to you as. Looks fine but feels of to me.

@dkijania dkijania merged commit 269a67d into MinaProtocol:main Mar 31, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants