Page MenuHomePhabricator

[native] Backup log upload
ClosedPublic

Authored by michal on Jan 15 2024, 6:52 AM.
Tags
None
Referenced Files
F2208898: D10633.id36156.diff
Sun, Jul 7, 3:24 PM
Unknown Object (File)
Sat, Jul 6, 12:46 AM
Unknown Object (File)
Thu, Jul 4, 1:06 PM
Unknown Object (File)
Sun, Jun 30, 2:26 PM
Unknown Object (File)
Sun, Jun 30, 12:15 AM
Unknown Object (File)
Thu, Jun 27, 6:02 AM
Unknown Object (File)
Thu, Jun 27, 4:32 AM
Unknown Object (File)
Wed, Jun 26, 3:24 PM
Subscribers

Details

Summary

ENG-5560 : Upload logic for logs

Similarly to the previous diff we want to upload log files to the backup service. One difference is that we need to wait for upload confirmation from the backup service, sent as a WS message, and only then remove the backup file. To prevent sending the same log files multiple times we keep track of logs that were sent but not yet confirmed in a HashSet that is shared between sender and receiver futures.

Depends on D10632

Test Plan

Create backup-123-log-456 and backup-123-log-456-attachments files. Enable backup upload. Make sure all data was successfuly uploaded to the backup service and the files were removed.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Some renames and code refactor to make it cleaner

This revision is now accepted and ready to land.Jan 18 2024, 11:34 PM