Page MenuHomePhabricator

Record command exit status on Execute objects in Phage
ClosedPublic

Authored by epriestley on Feb 19 2017, 2:58 PM.
Tags
None
Referenced Files
F14072165: D17386.diff
Wed, Nov 20, 7:43 PM
Unknown Object (File)
Sun, Nov 17, 3:23 AM
Unknown Object (File)
Sat, Oct 26, 4:34 PM
Unknown Object (File)
Oct 17 2024, 9:31 PM
Unknown Object (File)
Oct 16 2024, 5:05 AM
Unknown Object (File)
Sep 25 2024, 6:28 AM
Unknown Object (File)
Sep 20 2024, 8:49 PM
Unknown Object (File)
Sep 15 2024, 10:29 PM
Subscribers
None

Details

Summary

Ref T2794. With Phacility-specific changes elsewhere, this allows me to add some helpful summary output to the current phage UI.

I'm thinking about having a "plan" (a DOM-like document describing which commands to execute where) produce a "report" document, because a command may actually have multiple exit statuses (for example, if it was automatically retried after a failure). But this makes actually using phage to do things quite a bit easier for now.

Test Plan

Ran some cluster operations and, with changes elsewhere, got more useful high-level reporting about overall command state.

Diff Detail

Repository
rPHU libphutil
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Feb 19 2017, 5:34 PM
This revision was automatically updated to reflect the committed changes.