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.

Sign up for GitHub

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

Jump to bottom

Docker optional backup and restore with documentation #1692

Merged
merged 2 commits into from
Oct 22, 2023
Merged

Docker optional backup and restore with documentation #1692

merged 2 commits into from
Oct 22, 2023

Conversation

TrevorBenson
Copy link
Collaborator

@TrevorBenson TrevorBenson commented Oct 15, 2023 •
edited
Loading

This branch makes the container's backup and restore logic optional.

  1. Adds variable ENABLE_BACKUP that when set to Y enables the backup at container start.
  2. Adds variable ENABLE_RESTORE that when set to Y enables the restore at container start.
  3. Changes the backup path from /opt/cardano/cnode/priv to /opt/cardano/cnode/backup.
  4. Includes Docker Tips documentation explaining the ways to enable the creation of backups and restoring a backup.

closes #1691

@TrevorBenson TrevorBenson marked this pull request as ready for review October 15, 2023 03:16
@TrevorBenson TrevorBenson self-assigned this Oct 15, 2023
@TrevorBenson TrevorBenson added documentation Improvements or additions to documentation enhancement New feature or request docker labels Oct 15, 2023
@rdlrt rdlrt merged commit 7a11683 into cardano-community:alpha Oct 22, 2023
1 check passed
@TrevorBenson TrevorBenson deleted the container-backup-and-restore-as-an-option branch November 28, 2023 06:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rdlrt rdlrt rdlrt approved these changes

@redoracle redoracle Awaiting requested review from redoracle

Assignees

@TrevorBenson TrevorBenson

Labels
docker documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Container issues: 2x storage required and complicating db resets
2 participants
@TrevorBenson @rdlrt