Page MenuHomePhabricator

Daemons - handle daemons that can't be killed a bit better
ClosedPublic

Authored by btrahan on Aug 29 2014, 5:36 PM.
Tags
None
Referenced Files
F15489930: D10386.diff
Fri, Apr 11, 1:28 PM
F15482974: D10386.id25001.diff
Wed, Apr 9, 8:53 AM
F15474387: D10386.id.diff
Sun, Apr 6, 8:20 AM
F15470991: D10386.diff
Sat, Apr 5, 2:49 AM
F15444630: D10386.id.diff
Thu, Mar 27, 10:27 AM
F15368493: D10386.id25045.diff
Mar 12 2025, 2:30 AM
F15352646: D10386.diff
Mar 10 2025, 5:32 PM
Unknown Object (File)
Feb 22 2025, 1:39 PM
Subscribers

Details

Summary

Ref T2374. Fixes T5988.

Keep track of what's been killed and not been killed, and surface that maybe you need sudo if things don't get killed with --force

...also basically make this force thing work. I managed to convinced myself stuff was getting killed with --force when it mostly wasn't. Make sure the --force parameter gets pushed as low as it needs to go to have things get killed.

Test Plan
  • sudo ./bin/phd restart
  • rm -rf /var/tmp/phd/pid/*
  • ./bin/phd stop --> get warning about rogue daemons
  • ./bin/phd stop X --> get warning about no running daemons
  • ./bin/phd stop --force --> get warning about not being able to kill daemons
  • sudo ./bin/phd stop --force --> kill daemons successfully

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan retitled this revision from to Daemons - handle daemons that can't be killed a bit better.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Sep 3 2014, 9:24 PM
btrahan updated this revision to Diff 25045.

Closed by commit rP7ecbc707846f (authored by @btrahan).