Opened 6 years ago

Closed 5 years ago

#2639 closed enhancement (fixed)

Support 'rebuild'

Reported by: dustin Owned by: tardyp
Priority: major Milestone: 0.9.0
Version: Keywords:
Cc:

Description (last modified by dustin)

The BuilderControl rebuildBuild method allowed users to request a rebuild of a build.

That class and method are gone, but need to be replaced with a Data API control method that can be called from the Web UI.

Change History (9)

comment:1 Changed 6 years ago by tardyp

The implementation should be done differently that 0.8, and do more what people are expecting, i.e exactly duplicate the buildrequest.

Probably we could keep the same buildset.

The previous way of doing things re-use the build properties, which can be overwritten, and thus be different at the end of the build, and at the start. Thus the rebuild will not be exact.

comment:2 Changed 5 years ago by dustin

  • Description modified (diff)
  • Summary changed from [nine] Support 'rebuild' to Support 'rebuild'

The present implementation in 0.8.x offers a choice of using the updated build properties, or the buildset properties. I'm not tied to that -- I'd be happy to only support the latter.

This is a little complicated by the fact that webhead masters don't necessarily have write access to the database, so all they can do in response to a button clicked in the UI is to send a message.

So this will need to use the same cross-master request mechanism as the force scheduler, and will need some component capable of issuing rebuilds. It makes the most sense, I think, to implement that as another scheduler like the ForceScheduler?, but with fewer options.

comment:3 Changed 5 years ago by tardyp

I think we should implement it fist as if webstatus have write access to db, and when we have proper RPC baked in mq, we can implement it "properly"

comment:5 Changed 5 years ago by dustin

We'll keep this one; I closed #3158.

comment:6 Changed 5 years ago by dustin

  • Owner set to shenal
  • Status changed from new to assigned

comment:8 Changed 5 years ago by tardyp

  • Owner changed from shenal to tardyp

comment:9 Changed 5 years ago by tardyp

  • Resolution set to fixed
  • Status changed from assigned to closed

fixed

Note: See TracTickets for help on using tickets.