Skip to content

Add volume mount for geth data#49

Merged
mdehoog merged 3 commits intobase:mainfrom
Freekers:main
Jun 15, 2023
Merged

Add volume mount for geth data#49
mdehoog merged 3 commits intobase:mainfrom
Freekers:main

Conversation

@Freekers
Copy link
Copy Markdown
Contributor

@Freekers Freekers commented May 4, 2023

Currently geth data is stored within the container. This means all chaindata is lost whenever the container is destroyed, for example when upgrading to the latest version using docker-compose down instead of docker-compose stop. This can be prevented by volume mounting the data directory to a directory on the host machine, which this PR proposes.

@wbnns
Copy link
Copy Markdown
Contributor

wbnns commented May 5, 2023

@Freekers

Heya! Thanks for the suggestion here and welcome to the project!

From my POV, I definitely see where you are coming from, and yeah, if one removes the container that is a long time to restore the chain data. Maybe one concern (possibly others) is that this separates the data from the container, and leaves an artifact/remnant once the container is removed (a big one).

But let's see what others think! 👍

Side note, but tangential: we're working on making snapshots of chain data available to reduce sync time required in the event the container is removed and/or needs to be restored.

Michael de Hoog added 2 commits June 15, 2023 12:56
@cb-heimdall
Copy link
Copy Markdown
Collaborator

Review Error for roberto-bayardo @ 2023-06-15 23:40:28 UTC
User failed mfa authentication, see go/mfa-help

@mdehoog mdehoog merged commit d889e70 into base:main Jun 15, 2023
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.

5 participants