Page MenuHomePhabricator

Enable backup creation and restoration from Rust
ClosedPublic

Authored by marcin on Jan 3 2024, 8:56 AM.
Tags
None
Referenced Files
F3379417: D10512.id35716.diff
Wed, Nov 27, 3:46 PM
F3379410: D10512.id35715.diff
Wed, Nov 27, 3:45 PM
F3379386: D10512.id35162.diff
Wed, Nov 27, 3:38 PM
F3379312: D10512.id35491.diff
Wed, Nov 27, 3:31 PM
F3379249: D10512.id35685.diff
Wed, Nov 27, 3:27 PM
F3378880: D10512.diff
Wed, Nov 27, 1:45 PM
Unknown Object (File)
Wed, Nov 13, 7:08 AM
Unknown Object (File)
Wed, Nov 13, 7:08 AM
Subscribers

Details

Summary

This differential provides boilerplate code so that we can call BackupOperationsExecutor from Rust.

Test Plan
  1. Comment line in DatabaseManager where we remove backup directory on log-out.
  2. Add call to create_main_compaction in restore_backup in backup.rs
  3. Add some logging to get encryption key.
  4. Create some drafts.
  5. Tap text backup restore protocol button.
  6. Log out.
  7. Replace call to create_main_compaction in restore_backup in backup.rs with a call to get_backup_file_path with previous backup id and restore_from_main_compaction with previous backup id and encryption key logged before log-out.
  8. Tap text backup restore protocol button.
  9. Restart the app. Ensure that drafts created before log-out are now present.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Jan 3 2024, 9:32 AM
Harbormaster failed remote builds in B25453: Diff 35162!

LGTM but @michal needs to confirm as this is strongly connected to his work

This revision is now accepted and ready to land.Jan 5 2024, 6:31 AM

This diff needs to be updated to expose getBackupDirectoryPath method to Rust.

Expose getBackupDirectoryPath to Rust.

This revision is now accepted and ready to land.Jan 10 2024, 5:42 AM
This revision was landed with ongoing or failed builds.Jan 17 2024, 3:16 AM
This revision was automatically updated to reflect the committed changes.