Home GnuPG

Clean up a few more daemon behaviors
c0bc116bedc8Unpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Clean up a few more daemon behaviors

Summary:
Ref T12298.

  • If all daemons in all pools are hibernating already, we may exit immediately without sending EXIT events and fail to update the web UI.
  • A max() should be min() ("sleep for at most 180 seconds", not "sleep for at least 180 seconds").
  • If there are no daemons in a pool (unlikely/theoretical), we might not set the shutdown flag on the pool correctly.
  • An autoscale message may actually mean that a pool is exiting.

Test Plan:

  • Ran bin/phd debug task, waited for daemon to hibernate, killed it, saw it vanish from daemon console.
  • Saw it hibernate for 60s, not 180s.
  • Uh, killed a pool real fast and nothing broke? This one is hard/impossible to test.
  • Read the newer autoscale message.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T12298

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

Details

Provenance
epriestley <git@epriestley.com>Authored on Mar 24 2017, 9:40 PM
Parents
rPHUTILf2b2abeacf84: Send EXIT events more consistently from daemons
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHUTILc0bc116bedc8: Clean up a few more daemon behaviors (authored by epriestley <git@epriestley.com>).Mar 24 2017, 10:01 PM