Page MenuHomePhabricator
Differential D20608

Remove "phd.pid-directory" configuration and stop passing "piddir" to daemons
ClosedPublic
Actions

Authored by epriestley on Jun 20 2019, 10:11 PM.
Tags
None
Referenced Files
F15333240: D20608.id49145.diff
Sat, Mar 8, 1:12 AM
F15332605: D20608.id.diff
Fri, Mar 7, 6:33 PM
Unknown Object (File)
Thu, Feb 27, 10:54 AM
Unknown Object (File)
Mon, Feb 24, 12:01 AM
Unknown Object (File)
Feb 10 2025, 5:25 AM
Unknown Object (File)
Feb 9 2025, 7:55 AM
Unknown Object (File)
Feb 9 2025, 7:55 AM
Unknown Object (File)
Feb 9 2025, 7:55 AM
View All Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T13321: "bin/phd" may try to kill processes it does not own if the daemon PID logfiles on disk are out of date
Commits
rP65bc481c91de: Remove "phd.pid-directory" configuration and stop passing "piddir" to daemons
Summary

Ref T13321. The daemons no longer write PID files, so we no longer need to pass any of this stuff to them.

Test Plan

Grepped for affected symbols.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Jun 20 2019, 10:11 PM
Harbormaster completed remote builds in B23017: Diff 49145.Jun 20 2019, 10:12 PM
epriestley requested review of this revision.Jun 20 2019, 10:12 PM
amckinley accepted this revision.Jun 20 2019, 10:52 PM
amckinley added inline comments.
src/applications/daemon/management/PhabricatorDaemonManagementWorkflow.php
151

Mission accomplished!

This revision is now accepted and ready to land.Jun 20 2019, 10:52 PM
Closed by commit rP65bc481c91de: Remove "phd.pid-directory" configuration and stop passing "piddir" to daemons (authored by epriestley). · Explain WhyJun 24 2019, 6:29 PM
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rP65bc481c91de: Remove "phd.pid-directory" configuration and stop passing "piddir" to daemons.

Revision Contents
Changeset List

PathSize
src/
applications/
config/
check/
3 lines
option/
4 lines
daemon/
management/
16 lines
infrastructure/
daemon/
control/
135 lines

Diff 49173

View Options

src/applications/config/check/PhabricatorExtraConfigSetupCheck.php

Loading...
View Options

src/applications/config/option/PhabricatorPHDConfigOptions.php

Loading...
View Options

src/applications/daemon/management/PhabricatorDaemonManagementWorkflow.php

Loading...
View Options

src/infrastructure/daemon/control/PhabricatorDaemonReference.php

Loading...
Phabricator · Built by Phacility