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

Branch

softparse

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 1420
Build 1420: [Placeholder Plan] Wait for 30 Seconds

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/

infrastructure/

daemon/

control/

PhabricatorDaemonReference.php

8 lines

Diff 23409

View Options

src/infrastructure/daemon/control/PhabricatorDaemonReference.php

Continue on bad daemon pid dataClosedPublicActions