HomePhabricator

Add behaviors to Build Plans: hold drafts, affect buildables, warn on landing…

Tags
None
Referenced Files
F6244830: Screen Shot 2019-02-26 at 9.36.03 AM.png
Mar 6 2019, 1:40 PM
F6244828: Screen Shot 2019-02-26 at 9.35.57 AM.png
Mar 6 2019, 1:40 PM
Subscribers
None
Tokens
"Burninate" token, awarded by jmeador.

Description

Add behaviors to Build Plans: hold drafts, affect buildables, warn on landing, restartable, runnable

Summary:
Depends on D20219. Ref T13258. Ref T11415. Installs sometimes have long-running builds or unimportant builds which they may not want to hold up drafts, affect buildable status, or warn during arc land.

Some builds have side effects (like deployment or merging) and are not idempotent. They can cause problems if restarted.

In other cases, builds are isolated and idempotent and generally safe, and it's okay for marketing interns to restart them.

To address these cases, add "Behaviors" to Build Plans:

  • Hold Drafts: Controls how the build affects revision promotion from "Draft".
  • Warn on Land: Controls the "arc land" warning.
  • Affects Buildable: Controls whether we care about this build when figuring out if a buildable passed or failed overall.
  • Restartable: Controls whether this build may restart or not.
  • Runnable: Allows you to weaken the requirements to run the build if you're confident it's safe to run it on arbitrary old versions of things.
NOTE: This only implements UI, none of these options actually do anything yet.

Test Plan:
Mostly poked around the UI. I'll actually implement these behaviors next, and vet them more thoroughly.

Screen Shot 2019-02-26 at 9.35.57 AM.png (283×1 px, 53 KB)

Screen Shot 2019-02-26 at 9.36.03 AM.png (693×837 px, 114 KB)

Reviewers: amckinley

Reviewed By: amckinley

Subscribers: PHID-OPKG-gm6ozazyms6q6i22gyam

Maniphest Tasks: T13258, T11415

Differential Revision: https://secure.phabricator.com/D20220