Page MenuHomePhabricator

Allow daemons to be terminated in the absence of MySQL
ClosedPublic

Authored by joshuaspence on Jan 14 2015, 11:41 AM.
Tags
None
Referenced Files
F13811352: D11385.diff
Wed, Sep 18, 3:29 AM
Unknown Object (File)
Fri, Sep 13, 3:19 PM
Unknown Object (File)
Fri, Sep 13, 3:18 PM
Unknown Object (File)
Fri, Sep 13, 3:18 PM
Unknown Object (File)
Fri, Sep 13, 3:17 PM
Unknown Object (File)
Sun, Sep 1, 7:35 AM
Unknown Object (File)
Tue, Aug 27, 9:55 PM
Unknown Object (File)
Sat, Aug 24, 10:10 PM
Subscribers

Details

Summary

Fixes T6842. Allow the daemons to always be terminated, even if MySQL is down. I was hoping to be able to optionally enable this behavior with the --force flag, but this seems messy.

Test Plan
> ./bin/phd start
Freeing active task leases...
Freed 1 task lease(s).
Preparing to launch daemons.
NOTE: Logs will appear in '/var/tmp/phd/log/daemons.log'.

Starting daemons as phd
Launching daemon "PhabricatorRepositoryPullLocalDaemon".
Starting daemons as phd
Launching daemon "PhabricatorGarbageCollectorDaemon".
Starting daemons as phd
Launching daemon "PhabricatorTaskmasterDaemon".
Done.

> service mysql stop
mysql stop/waiting

> ./bin/phd stop
Interrupting daemon 'PhabricatorRepositoryPullLocalDaemon' (4263)...
Interrupting daemon 'PhabricatorGarbageCollectorDaemon' (4271)...
Interrupting daemon 'PhabricatorTaskmasterDaemon' (4287)...
Daemon 4263 exited.
Daemon 4271 exited.
Daemon 4287 exited.

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 3835
Build 3847: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence retitled this revision from to Allow daemons to be terminated in the absence of MySQL.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jan 14 2015, 2:00 PM
This revision was automatically updated to reflect the committed changes.