Page MenuHomePhabricator

[services] Backup - Create new backup - avoid suppressing database errors
ClosedPublic

Authored by karol on May 19 2022, 3:01 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 20, 6:37 AM
Unknown Object (File)
Fri, Nov 29, 11:06 PM
Unknown Object (File)
Nov 15 2024, 10:34 PM
Unknown Object (File)
Nov 5 2024, 3:23 PM
Unknown Object (File)
Oct 13 2024, 2:01 PM
Unknown Object (File)
Oct 13 2024, 2:01 PM
Unknown Object (File)
Oct 13 2024, 2:00 PM
Unknown Object (File)
Oct 13 2024, 1:56 PM

Details

Summary

Depends on D4077

When creating a new backup I mistakenly just printed out database errors to stdout on the server. That led to an unwanted situation when something went wrong with the database and the client received a positive outcome. With this change, the error breaks uploading a new backup and the client receives proper information on what went wrong.

Test Plan

Services can still be built.

It's hard to really test this. In order to do this, we'd have to intentionally hardcode invalid instructions on the database and then see the result on the client-side.

You can do that, please see D4074 on how to test the services with the client.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable