Paths

Table of Contentst

Differential D9749

Continue on bad daemon pid data
ClosedPublic
Actions

Authored by epriestley on Jun 26 2014, 10:17 PM.

Details

Reviewers

joshuaspence

Commits

Restricted Diffusion Commit
rP950d81dac066: Continue on bad daemon pid data

Summary

The rest of this code works if we hand off array(), and fataling here, while more correct, is harder for users to get out of (they have to go manually remove files) and not obvious.

Test Plan

Corrupted pid file and ran phd stop.

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 23409.Jun 26 2014, 10:17 PM

epriestley retitled this revision from to Continue on bad daemon pid data.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: joshuaspence.

Herald added a subscriber: epriestley. · View Herald TranscriptJun 26 2014, 10:17 PM

joshuaspence accepted this revision.Jun 26 2014, 10:18 PM

joshuaspence edited edge metadata.

This revision is now accepted and ready to land.Jun 26 2014, 10:18 PM

Harbormaster completed remote builds in B1420: Diff 23409.Jun 26 2014, 10:18 PM

Also recover from bad explicitArgv when running phd status.

Closed by commit rP950d81dac066 (authored by @epriestley).

Harbormaster completed remote builds in B1421: Diff 23410.Jun 26 2014, 10:22 PM

Revision Contents
Changeset List

Path

Size

src/

applications/

daemon/

storage/

PhabricatorDaemonLog.php

10 lines

infrastructure/

daemon/

control/

PhabricatorDaemonReference.php

8 lines

Diff 23411

View Options

src/applications/daemon/storage/PhabricatorDaemonLog.php