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)
Sat, May 31, 10:58 AM
Unknown Object (File)
Thu, May 29, 3:17 PM
Unknown Object (File)
Wed, May 28, 7:58 PM
Unknown Object (File)
May 9 2025, 6:57 AM
Unknown Object (File)
May 8 2025, 1:28 AM
Unknown Object (File)
Apr 27 2025, 3:12 PM
Unknown Object (File)
Apr 26 2025, 7:31 PM
Unknown Object (File)
Apr 24 2025, 11:07 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.