Add workaround docs for MemoryError during backup #4330
Conversation
eloquence
requested review from
emkll,
kushaldas and
redshiftzero
as code owners
docs/includes/backup-warning.txt
Outdated
| @@ -0,0 +1,19 @@ | |||
| .. note:: | |||
| When dealing with larger backups, the ``securedrop-admin backup`` command may | |||
| fail with a MemoryError at this stage of the operation: "Fetch the backup | |||
There was a problem hiding this comment.
Nit: Backticks around "MemoryError" would be more consistent with existing style.
eloquence
force-pushed
the
docs-backup-error
branch
from
b225498 to
902e4c2
Compare
There was a problem hiding this comment.
Followed @rmol 's excellent instructions from #4326 (comment)
- created a file in
/var/lib/securedrop/store - observe failure when performing
./securedrop-admin backup:
TASK [backup : Fetch the backup tarball back to the Admin Workstation.] ********
An exception occurred during task execution. To see the full traceback, use -vvv. The error was: MemoryError
- followed workaround instructions and confirmed the backup is copied and the file is present in the backup (however there might be a mistake in the destination directory?)
docs/includes/backup-warning.txt
Outdated
|
|
||
| .. code:: bash | ||
|
|
||
| rsync -av --progress --partial app:$(ssh app ls -1rt /tmp/sd-backup* | tail -1) install_files/ansible_base/ |
There was a problem hiding this comment.
is there a reason why it's ansible_base? Backups are normally install_files/ansible-base
There was a problem hiding this comment.
That is a typo that I didn't catch. It should be ansible-base.
There was a problem hiding this comment.
ah good catch, I'm guessing that's a hyphen/underscore typo in the original command
There was a problem hiding this comment.
It is. You transcribed it accurately.
There was a problem hiding this comment.
We didn't catch it during the upgrade because we ended up using rsync to transfer the backup back to the app server.
Time-sensitive due to Xenial updates in progress. This will be resolved with #4326 in 0.12.1, though we may want to keep some version of this documentation around in case people still encounter issues during the playbook run. Credit to @rmol for authoring the rsync command used here, which has been used during real-world upgrades.
eloquence
force-pushed
the
docs-backup-error
branch
from
902e4c2 to
8571ee1
Compare
Time-sensitive due to Xenial updates in progress.
This will be resolved with #4326 in 0.12.1, though we may want to
keep some version of this documentation around in case people
still encounter issues during the playbook run.
Credit to @rmol for authoring the rsync command used here, which
has been used during real-world upgrades.
Status
Ready for review
Checklist
make docs-lint) passed locally