Paths

Table of Contentst

Don't SIGINT ourselves on bad daemon data
ClosedPublic
Actions

Authored by epriestley on Aug 20 2014, 5:34 PM.

Tags

None

Referenced Files

	F15523127: D10308.diff
	Sun, Apr 20, 11:17 PM

	F15478555: D10308.id24817.diff
	Tue, Apr 8, 1:04 AM

	F15475951: D10308.id24821.diff
	Mon, Apr 7, 1:29 AM

	F15474880: D10308.id.diff
	Sun, Apr 6, 1:06 PM

	F15471868: D10308.diff
	Sat, Apr 5, 8:43 AM

	F15433241: D10308.id24821.diff
	Mar 24 2025, 10:14 PM

	F15397096: D10308.diff
	Mar 16 2025, 6:08 PM

	F15388449: D10308.id24821.diff
	Mar 15 2025, 3:50 AM

View All Files

Subscribers

epriestley

• mholden

Details

Reviewers

btrahan

Commits

Restricted Diffusion Commit
rP15385e1fe9b5: Don't SIGINT ourselves on bad daemon data

Summary

If daemon data is mangled, bin/phd restart will SIGINT process 0, which kills it.

uh oh T.T so sad

Test Plan

Used bin/phd start to start daemons; removed PID information from one; saw bin/phd stop shut down cleanly and not kill itself.

Diff Detail

Repository

rP Phabricator

Branch

sigint

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 2289
Build 2293: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 24817.Aug 20 2014, 5:34 PM

epriestley retitled this revision from to Don't SIGINT ourselves on bad daemon data.

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 TranscriptAug 20 2014, 5:34 PM

Harbormaster completed remote builds in B2289: Diff 24817.Aug 20 2014, 5:35 PM

I complained about this issue and I approve this commit ;p

btrahan accepted this revision.Aug 20 2014, 8:02 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Aug 20 2014, 8:02 PM

Closed by commit rP15385e1fe9b5 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

daemon/

management/

PhabricatorDaemonManagementWorkflow.php

6 lines

Diff 24817

View Options

src/applications/daemon/management/PhabricatorDaemonManagementWorkflow.php

Don't SIGINT ourselves on bad daemon dataClosedPublicActions