Page MenuHomePhabricator

Replace "Cancel Build" with "Stop", "Resume" and "Restart"
ClosedPublic

Authored by epriestley on Jan 5 2014, 7:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 19, 3:30 AM
Unknown Object (File)
Fri, Apr 19, 3:30 AM
Unknown Object (File)
Fri, Apr 19, 3:30 AM
Unknown Object (File)
Fri, Apr 19, 3:30 AM
Unknown Object (File)
Thu, Apr 11, 4:42 AM
Unknown Object (File)
Mon, Apr 8, 10:34 AM
Unknown Object (File)
Sat, Apr 6, 7:50 PM
Unknown Object (File)
Sat, Apr 6, 2:23 PM
Subscribers
Tokens
"The World Burns" token, awarded by demo.

Details

Summary

Ref T1049. Currently you can cancel a build, but now that we're tracking a lot more state we can stop, resume, and restart builds.

When the user issues a command against a build, I'm writing it into an auxiliary queue (HarbormasterBuildCommand) and then reading them out in the worker. This is mostly to avoid race messes where we try to save() the object in multiple places: basically, the BuildEngine is the only thing that writes to Build objects, and it holds a lock while it does it.

Test Plan
  • Created a plan which runs "sleep 2" a bunch of times in a row.
  • Stopped, resumed, and restarted it.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

We need icons for these? I have play, pause, stop, etc, icons in Glyphicons.

These are the icons we're using right now -- I think they're OK, we have better ones then by all means:

{F98538}

epriestley updated this revision to Unknown Object (????).Jan 6 2014, 8:31 PM
  • Use fancy new icons.