Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

hurdles restoring from snapshot via dockerized cardano-db-sync #691

Closed
edwmurph opened this issue Jul 12, 2021 · 7 comments
Closed

hurdles restoring from snapshot via dockerized cardano-db-sync #691

edwmurph opened this issue Jul 12, 2021 · 7 comments

Comments

@edwmurph
Copy link

  • cardano-db-tool is not currently baked into the image but it's necessary for doing snapshot restores. can cardano-db-tool be baked into the docker image going forward?
  • postgresql-setup.sh is required for restoring from the snapshot but i can't find it on the cardano-db-sync image. i could manually copy it over but it would be ideal if it was also baked into the image to facilitate setting up automation on my end for restoring from snapshots
@erikd
Copy link
Contributor

erikd commented Jul 13, 2021

cardano-db-tool is needed for snapshot creation, but not restoration.

@edwmurph
Copy link
Author

@erikd oh great! it would still be helpful though if cardano-db-tool was baked into the docker image for creating custom snapshots going forward but for the near term, just baking the postgresql-setup.sh script into the docker image would help a lot

@erikd
Copy link
Contributor

erikd commented Jul 14, 2021

I know close to zero about Docker, so I would love to get a fix for this (and instructions on how to test it). PRs gladly accepted.

@edwmurph
Copy link
Author

looks like you guys use nix so im not exactly sure how to implement this

@erikd
Copy link
Contributor

erikd commented Jul 14, 2021

OK, we have someone on it, but they may not get to it immediately.

@jbgi
Copy link
Contributor

jbgi commented Jul 27, 2021

initial implementation in #712
(still need some testing)

@erikd
Copy link
Contributor

erikd commented Jul 29, 2021

#712 has been merged to master and cherry picked into the release/10.0.x branch which will be released as 10.0.2.

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

No branches or pull requests

3 participants