Home GnuPG

Be less strict when detecting dead daemons
939fb69aa68bUnpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Be less strict when detecting dead daemons

Summary:
Fixes T12306. Currently, we warn about daemons not running even if they're in normal "alive" states, particularly "waiting to restart after a failure".

This check was made more strict in D12088, back when we tried to version check running daemons. Since we implemented auto-restart-after-config-change we don't do this anymore, so it should be fine to make this more lax again.

Test Plan:

  • Faked an exception for all tasks.
  • Before patch: reloading the daemon setup error sometimes raised a false positive ("waiting" daemon detected as dead).
  • After patch: daemon setup error no longer triggers.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T12306

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

Details

Provenance
epriestley <git@epriestley.com>Authored on Feb 22 2017, 11:05 PM
Parents
rPHAB6f50729a9171: Update Phabricator for new daemon pool changes
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHAB939fb69aa68b: Be less strict when detecting dead daemons (authored by epriestley <git@epriestley.com>).Feb 22 2017, 11:11 PM