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

Copy backup to Admin Workstation before manual noble migration #7441

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

legoktm
Copy link
Member

@legoktm legoktm commented Feb 14, 2025

Status

Work in progress

Description of Changes

If the admin has manually initiated a backup, let's copy a backup to the Admin Workstation so it's easier to use. The backup is automatically deleted at the end of a successful playbook run.

Especially for instances using SSH-over-Tor, this step may take a while, but if something does go wrong, it'll be nice.

A backup is still taken on the server that stays on the server, there's no adverse effect except for a slight delay in waiting for the extraneous backup.

Testing

How should the reviewer test this PR?

  • Start manual migration of the app server
  • See debug message that the backup finished
  • Verify there's a backup in your securedrop directory
  • Wait for migration to finish
  • See that backup was deleted

Deployment

Any special considerations for deployment? n/a

Checklist

  • Linting and tests (make -C admin test) pass in the admin development container
  • Configuration tests pass

If the admin has manually initiated a backup, let's copy a backup to the
Admin Workstation so it's easier to use. The backup is automatically
deleted at the end of a successful playbook run.

Especially for instances using SSH-over-Tor, this step may take a while,
but if something does go wrong, it'll be nice.

A backup is still taken on the server that stays on the server, there's
no adverse effect except for a slight delay in waiting for the
extraneous backup.
@legoktm legoktm added the noble Ubuntu Noble related work label Feb 14, 2025
@legoktm legoktm marked this pull request as ready for review February 14, 2025 21:36
@legoktm legoktm requested a review from a team as a code owner February 14, 2025 21:36
@legoktm
Copy link
Member Author

legoktm commented Feb 14, 2025

This is submitted without any testing or verifying of the ansible syntax - I'll try to do that early next week.

@legoktm legoktm added this to the SecureDrop 2.12.0 milestone Feb 14, 2025
@legoktm
Copy link
Member Author

legoktm commented Feb 18, 2025

At this point I'd rather not pull this in and just have people manually start a backup on their own. If they don't, we still have the on-server backup.

@legoktm legoktm removed this from the SecureDrop 2.12.0 milestone Feb 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
noble Ubuntu Noble related work
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

1 participant