Page MenuHomePhabricator
Differential D17463

Support timeouts for Phage commands
ClosedPublic
Actions

Authored by epriestley on Mar 4 2017, 6:11 PM.
Tags
None
Referenced Files
F19520675: D17463.diff
Fri, Jan 16, 3:36 PM
F19103005: D17463.id.diff
Dec 5 2025, 8:16 AM
F19100469: D17463.diff
Dec 4 2025, 11:14 PM
F19072415: D17463.id41995.diff
Nov 30 2025, 10:24 PM
F19063388: D17463.id.diff
Nov 29 2025, 6:23 PM
F18902691: D17463.id41996.diff
Nov 8 2025, 2:16 AM
F18902690: D17463.id41995.diff
Nov 8 2025, 2:16 AM
F18902689: D17463.id.diff
Nov 8 2025, 2:16 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T2794: Implement Phage (like Hypershell)
Commits
rPHU796cb1c2ee27: (stable) Promote 2017 Week 9
rPHU13a200ca7621: Support timeouts for Phage commands
Summary

Ref T2794. Occasionally stuff hangs for whatever reason. Provide a crude per-command timeout as a general tool for dealing with this.

Test Plan

Added some sleep and ran with --timeout, saw commands get killed and reported as failed.

Diff Detail

Repository
rPHU libphutil
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Mar 4 2017, 6:11 PM
chad accepted this revision.Mar 4 2017, 6:11 PM
This revision is now accepted and ready to land.Mar 4 2017, 6:11 PM
Harbormaster completed remote builds in B15899: Diff 41995.Mar 4 2017, 6:11 PM
Closed by commit rPHU13a200ca7621: Support timeouts for Phage commands (authored by epriestley, committed by epriestley). · Explain WhyMar 4 2017, 6:15 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
phage/
__tests__/
2 lines
action/
2 lines
21 lines
agent/
7 lines

Diff 41996

View Options

src/phage/__tests__/PhageAgentTestCase.php

Loading...
View Options

src/phage/action/PhageAgentAction.php

Loading...
View Options

src/phage/action/PhageExecuteAction.php

Loading...
View Options

src/phage/agent/PhagePHPAgent.php

Loading...
Phabricator · Built by Phacility