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
F3505113: D10602.id35671.diff
Fri, Dec 20, 12:15 PM
Unknown Object (File)
Thu, Nov 21, 3:58 AM
Unknown Object (File)
Thu, Nov 21, 3:58 AM
Unknown Object (File)
Oct 23 2024, 2:46 AM
Unknown Object (File)
Oct 22 2024, 1:06 PM
Unknown Object (File)
Oct 22 2024, 9:00 AM
Unknown Object (File)
Oct 22 2024, 8:37 AM
Unknown Object (File)
Oct 22 2024, 8:37 AM
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