Home GnuPG

Send EXIT events more consistently from daemons
f2b2abeacf84Unpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Send EXIT events more consistently from daemons

Summary:
Ref T12298. When we bin/phd restart daemons, we currently may miss sending EXIT events for daemons which aren't actively running at the time, since the EXIT event only happens when the subprocess exits, not if we tear down a waiting/sleeping/hibernating daemon.

The most common case of this is hibernating daemons. A less common case is daemons which have exited because of an error (usually, taskmasters).

In these cases, the web UI doesn't get cleaned up properly. Instead, send EXIT from any kind of teardown so we knock daemons out of the web UI more often.

Test Plan: Ran bin/phd restart a bunch, saw fewer hanging daemons in the web UI at /daemon/.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T12298

Differential Revision: https://secure.phabricator.com/D17553

Details

Provenance
epriestley <git@epriestley.com>Authored on Mar 24 2017, 4:49 PM
Parents
rPHUTILbebe54d5762f: Don't awaken or scale pools during daemon shutdown
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHUTILf2b2abeacf84: Send EXIT events more consistently from daemons (authored by epriestley <git@epriestley.com>).Mar 24 2017, 5:19 PM