Page MenuHomePhabricator
Differential D16053

Log when daemons trigger a reload in response to config changes
ClosedPublic
Actions

Authored by epriestley on Jun 6 2016, 1:34 PM.
Tags
None
Referenced Files
F14917761: D16053.id38626.diff
Thu, Feb 13, 11:23 AM
Unknown Object (File)
Sun, Feb 9, 1:44 AM
Unknown Object (File)
Sun, Feb 9, 1:44 AM
Unknown Object (File)
Tue, Feb 4, 10:13 PM
Unknown Object (File)
Wed, Jan 29, 3:45 AM
Unknown Object (File)
Fri, Jan 24, 4:00 PM
Unknown Object (File)
Thu, Jan 23, 3:41 PM
Unknown Object (File)
Thu, Jan 23, 3:41 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T10811: `phd-daemon` crash after automatic restart following config change
Commits
rPHU5e43755a5b58: Log when daemons trigger a reload in response to config changes
Summary

Ref T10811. This puts an explicit message in the log when a module triggers a reload.

  • Also, lift declare(ticks = 1) up to top level.
  • Make sure verbose messages hit the disk log, not just the database log.
Test Plan

Ran daemons, changed config, saw them reload automatically with an explicit message.

Diff Detail

Repository
rPHU libphutil
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 38626.Jun 6 2016, 1:34 PM
epriestley retitled this revision from to Log when daemons trigger a reload in response to config changes.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T10811: `phd-daemon` crash after automatic restart following config change.
epriestley mentioned this in D16054: Lift ticks declaration to top level in Phabricator daemons.
chad accepted this revision.Jun 6 2016, 1:36 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Jun 6 2016, 1:36 PM
Closed by commit rPHU5e43755a5b58: Log when daemons trigger a reload in response to config changes (authored by epriestley, committed by epriestley). · Explain WhyJun 6 2016, 1:39 PM
This revision was automatically updated to reflect the committed changes.
epriestley mentioned this in rP0bf337b06c1f: Lift ticks declaration to top level in Phabricator daemons.

Revision Contents
Changeset List

PathSize
scripts/
daemon/
2 lines
src/
daemon/
15 lines
14 lines

Diff 38628

View Options

scripts/daemon/launch_daemon.php

Loading...
View Options

src/daemon/PhutilDaemonHandle.php

Loading...
View Options

src/daemon/PhutilDaemonOverseer.php

Loading...
Phabricator · Built by Phacility