Page MenuHomePhabricator

[native] Backup log upload
ClosedPublic

Authored by michal on Jan 15 2024, 6:52 AM.
Tags
None
Referenced Files
F2902766: D10633.diff
Sat, Oct 5, 6:53 PM
Unknown Object (File)
Sat, Sep 7, 2:20 PM
Unknown Object (File)
Sep 5 2024, 3:06 PM
Unknown Object (File)
Sep 5 2024, 3:06 PM
Unknown Object (File)
Sep 5 2024, 3:06 PM
Unknown Object (File)
Sep 5 2024, 3:06 PM
Unknown Object (File)
Sep 5 2024, 3:06 PM
Unknown Object (File)
Sep 5 2024, 3:05 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
No Lint Coverage
Unit
No Test Coverage

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