Page MenuHomePhabricator

If the overseer can't update the PID file, just move on
ClosedPublic

Authored by epriestley on Jun 19 2017, 9:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 1, 6:21 PM
Unknown Object (File)
Mon, Apr 1, 3:08 PM
Unknown Object (File)
Sun, Mar 31, 10:49 AM
Unknown Object (File)
Mar 18 2024, 2:59 AM
Unknown Object (File)
Feb 27 2024, 9:31 AM
Unknown Object (File)
Feb 21 2024, 10:09 AM
Unknown Object (File)
Feb 21 2024, 9:24 AM
Unknown Object (File)
Feb 10 2024, 1:51 AM
Subscribers
None

Details

Summary

Ref T12857. This is a little murky, but this behavior seems clearly better.

When the disk is full, daemons may attempt to update their PID files and fail. When this happens, keep running rather than exiting.

Test Plan
  • Made Filesystem::writeFile() throw an exception unconditionally to simulate a full disk.
  • Ran bin/phd start.
  • Saw daemons exit immediately.
  • Applied patch, saw daemons stick around and do useful work instead.

Diff Detail

Repository
rPHU libphutil
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Jun 20 2017, 3:39 AM
This revision was automatically updated to reflect the committed changes.