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

Update backup/restore docs with sd-gpg, sd-app backup info #246

Merged
merged 1 commit into from
Jul 16, 2024

Conversation

rocodes
Copy link
Contributor

@rocodes rocodes commented Jul 16, 2024

Update backup/restore guide with information about backing up sd-app and sd-gpg

(Will clean up and mark ready shortly)

@rocodes rocodes requested a review from nathandyer July 16, 2024 12:45
@rocodes rocodes force-pushed the update-backup-restore-guide branch 2 times, most recently from 01da496 to 24800fb Compare July 16, 2024 15:21
docs/admin/reference/backup.rst Outdated Show resolved Hide resolved
docs/admin/reference/backup.rst Outdated Show resolved Hide resolved
docs/admin/reference/backup.rst Show resolved Hide resolved
@rocodes rocodes force-pushed the update-backup-restore-guide branch 3 times, most recently from f5ac4b1 to 7fac1fd Compare July 16, 2024 18:38
@rocodes rocodes marked this pull request as ready for review July 16, 2024 18:39
@rocodes
Copy link
Contributor Author

rocodes commented Jul 16, 2024

hey @deeplow, thanks for your comments - the PR is now ready for review, and I think I've addressed your changes :)

@rocodes rocodes requested a review from a team July 16, 2024 18:41
Copy link
Contributor

@nathandyer nathandyer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks so much for this @rocodes! Left a few notes about a couple things to tweak, but this is in good shape and I would feel comfortable landing this later today.

docs/admin/reference/backup.rst Outdated Show resolved Hide resolved
docs/admin/reference/backup.rst Outdated Show resolved Hide resolved
docs/admin/reference/backup.rst Outdated Show resolved Hide resolved
docs/admin/reference/backup.rst Outdated Show resolved Hide resolved
docs/admin/reference/backup.rst Show resolved Hide resolved
revise backup policy:

* Back up dom0 directly, instead of copying dom0 configration files to
  the vault VM.

* Include /etc/qubes and /etc/qubes-rpc in dom0 backup (by copying to
  home directory).

* Explain template upgrade for customized VMs and link to upstream docs.

* Include sd-gpg secret keys in backup. For most instances, this will be
  redundant, but for those that have imported an additional (legacy)
  Submission Private Key, this step is required.

* Include instructions for optionally backing up and restoring sd-app.
  This will allow users to skip the first sync on restoring.

* Include caveats and instructions regarding backing up of sensitive
  information.
@rocodes rocodes force-pushed the update-backup-restore-guide branch from 7fac1fd to b2c08c5 Compare July 16, 2024 20:34
@rocodes
Copy link
Contributor Author

rocodes commented Jul 16, 2024

Hey @nathandyer thanks for the detailed feedback, and sorry for my late-night typos. Changes addressed in b2c08c5

@rocodes rocodes requested a review from nathandyer July 16, 2024 20:35
Copy link
Contributor

@nathandyer nathandyer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the quick turnaround for those changes! This LGTM, going to go ahead and merge.

@nathandyer nathandyer added this pull request to the merge queue Jul 16, 2024
Merged via the queue into main with commit 6354343 Jul 16, 2024
4 checks passed
@nathandyer nathandyer deleted the update-backup-restore-guide branch July 16, 2024 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants