Page MenuHomePhabricator
Differential D11449

When storage is not initialized, write the error message to stderr instead of stdout
ClosedPublic
Actions

Authored by epriestley on Jan 20 2015, 10:02 PM.
Tags
None
Referenced Files
F14007304: D11449.diff
Tue, Oct 29, 3:25 AM
F13968337: D11449.id.diff
Oct 16 2024, 8:17 PM
F13965755: D11449.id27524.diff
Oct 16 2024, 4:07 AM
F13963704: D11449.diff
Oct 15 2024, 5:23 PM
F13962528: D11449.id27522.diff
Oct 15 2024, 10:39 AM
Unknown Object (File)
Oct 7 2024, 1:22 AM
Unknown Object (File)
Sep 25 2024, 9:17 PM
Unknown Object (File)
Sep 17 2024, 7:44 AM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
Commits
Restricted Diffusion Commit
rPe90695fc8332: When storage is not initialized, write the error message to stderr instead of…
Summary

We have to do some garbage nonsense to write database backups right now, see T6996.

When storage isn't initialized, we previously ended up with this message gzipped in a file and an empty error. Make the behavior slightly more tolerable.

Test Plan

Saw a meaningful error after trying to back up an uninitialized database.

Diff Detail

Repository
rP Phabricator
Branch
need2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 3950
Build 3963: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 27522.Jan 20 2015, 10:02 PM
epriestley retitled this revision from to When storage is not initialized, write the error message to stderr instead of stdout.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
Herald added a subscriber: epriestley. · View Herald TranscriptJan 20 2015, 10:02 PM
btrahan accepted this revision.Jan 20 2015, 10:04 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Jan 20 2015, 10:04 PM
Closed by commit rPe90695fc8332: When storage is not initialized, write the error message to stderr instead of… (authored by epriestley, committed by epriestley). · Explain WhyJan 20 2015, 10:14 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
infrastructure/
storage/
management/
workflow/
12 lines

Diff 27522

View Options

src/infrastructure/storage/management/workflow/PhabricatorStorageManagementDumpWorkflow.php

Loading...
Phabricator · Built by Phacility