Page MenuHomePhabricator

[backup] Allow one websocket connection to handle multiple backups
ClosedPublic

Authored by michal on Jan 11 2024, 7:34 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Sep 7, 9:56 AM
Unknown Object (File)
Sat, Sep 7, 9:25 AM
Unknown Object (File)
Sat, Sep 7, 9:24 AM
Unknown Object (File)
Sat, Sep 7, 9:24 AM
Unknown Object (File)
Fri, Sep 6, 10:33 PM
Unknown Object (File)
Aug 17 2024, 6:45 PM
Unknown Object (File)
Aug 15 2024, 4:31 PM
Unknown Object (File)
Aug 11 2024, 5:33 PM
Subscribers

Details

Summary

After working on a final backup client for native I realized that the backup service log API would be better if it wasn't opened per backup, and instead the messages contained the information which backup they apply to. This allow us to to run only one websocket connection on the backup client and handle all edge cases (e.g. there was no network connection, but we started creating a new backup and now have logs for two backups). This simplifies the backup client code a lot because we don't need to spawn new webosocket connection for every backup_id and keep track of when to kill them. This is a breaking change but backup service isn't used in production app (there is only staff-only opt-in testing code) so it should be fine.

Test Plan

Run integration tests.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage