Page MenuHomePhabricator

Make the new Build Plan behavior "Restartable" work
ClosedPublic

Authored by epriestley on Feb 28 2019, 8:05 PM.
Tags
None
Referenced Files
F19314345: D20230.id48337.diff
Thu, Dec 25, 2:18 AM
F19082934: D20230.id48337.diff
Dec 2 2025, 12:27 PM
F19040724: D20230.diff
Nov 26 2025, 1:29 PM
F18993161: D20230.id.diff
Nov 19 2025, 6:03 AM
F18985055: D20230.id48290.diff
Nov 17 2025, 12:52 PM
F18850526: D20230.diff
Oct 30 2025, 5:37 PM
F18820267: D20230.diff
Oct 22 2025, 1:42 PM
F18819789: D20230.id.diff
Oct 22 2025, 9:36 AM
Subscribers
Restricted Owners Package

Details

Summary

Ref T13258. Implements the "Restartable" behavior, to control whether a build may be restarted or not.

This is fairly straightforward because there are already other existing reasons that a build may not be able to be restarted.

Test Plan

Restarted a build. Marked it as not restartable, saw "Restart" action become disabled. Tried to restart it anyway, got a useful error message.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Owners added a subscriber: Restricted Owners Package.Feb 28 2019, 8:05 PM

Does anyone want an option for "only allow builds that have failed to be restarted"? For cases where builds are idempotent except for a final irrevocable step like "find servers running any version of the code != this version and kill them"?

This revision is now accepted and ready to land.Mar 5 2019, 7:47 PM

I haven't seen any of that so far, but it seems pretty reasonable to me. Maybe I'll just add it unless I can come up with some reason why it's secretly bad in the time it takes to implement.

This revision was automatically updated to reflect the committed changes.