Page MenuHomePhabricator

Separate individual daemon process logic into PhutilDaemonHandle
ClosedPublic

Authored by epriestley on Feb 22 2015, 3:19 PM.
Tags
None
Referenced Files
F12844817: D11851.id.diff
Thu, Mar 28, 11:50 PM
Unknown Object (File)
Wed, Mar 20, 6:48 PM
Unknown Object (File)
Feb 23 2024, 10:57 PM
Unknown Object (File)
Feb 12 2024, 2:03 PM
Unknown Object (File)
Feb 7 2024, 9:01 PM
Unknown Object (File)
Feb 1 2024, 5:38 AM
Unknown Object (File)
Jan 19 2024, 10:05 PM
Unknown Object (File)
Jan 19 2024, 10:04 PM
Subscribers

Details

Summary

Ref T7352. Overall, I want to run one overseer per daemon group instead of one overseer per daemon, to reduce the minimum footprint of an inactive Phacility instance.

PhutilDaemonOverseer currently has a mixture of logic specific to the overseer (e.g., argument parsing, signal handling) and logic specific to the daemon process (e.g., heartbeat handling).

Separate this logic, so the Overseer has a list of DaemonHandles. Currently, it always has exactly one DaemonHandle, but the logic anticipates a future diff where it may have more than one.

Test Plan
  • Ran bin/phd debug nice, bin/phd debug taskmaster, observed normal operation.
  • Used ^C and other signals to terminate daemons.
  • Viewed daemon status in daemon console.
  • Used phd start, phd stop to start and stop daemons normally.

Diff Detail

Repository
rPHU libphutil
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Separate individual daemon process logic into PhutilDaemonHandle.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Feb 23 2015, 5:22 PM