1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-23 07:12:41 +01:00
phorge-phorge/src/docs
epriestley 6219d30f6b Recommend dumping database backups with "--compress --output X" instead of "| gzip > X"
Summary:
Fixes T13304. Shell pipes and redirects do not have robust behavior when errors occur. We provide "--compress" and "--output" flags as robust alternatives, but do not currently recommend their use.

  - Recommend their use, since their error handling behavior is more robust in the face of issues like full disks.
  - If "--compress" is provided but won't work because the "zlib" extension is missing, raise an explicit error. I believe this extension is very common and this error should be rare. If that turns out to be untrue, we could take another look at this.
  - Also, verify some flag usage sooner so we can exit with an error faster if you mistype a "bin/storage dump" command.

Test Plan: Read documentation, hit affected error cases, did a dump and spot-checked that it came out sane looking.

Reviewers: amckinley

Reviewed By: amckinley

Maniphest Tasks: T13304

Differential Revision: https://secure.phabricator.com/D20572
2019-06-18 11:37:14 -07:00
..
book Merge "Audit" more completely into "Diffusion" 2017-01-11 16:28:42 -08:00
contributor Fix spelling 2017-10-09 10:48:04 -07:00
flavor Remove "Large Changes" documentation and make some minor behavioral improvements 2018-04-05 06:40:46 -07:00
tech Remove all "Phabricator Bot" code 2017-04-21 12:48:35 -07:00
user Recommend dumping database backups with "--compress --output X" instead of "| gzip > X" 2019-06-18 11:37:14 -07:00