Ticket #56 (closed enhancement: fixed)

Opened 3 years ago

Last modified 23 months ago

custom properties for triggerable schedulers

Reported by: dustin Owned by: warner
Priority: major Milestone: 0.7.8
Version: Keywords:
Cc: gerg.ward+buildbot@…, bhearsum@…

Description

Discussion was here:

 http://article.gmane.org/gmane.comp.python.buildbot.devel/2064

This basically provides an alternative to the dependent scheduler, allowing more sophisticated scheduling of inter-dependent builders.

Attachments

trigger.patch Download (14.1 KB) - added by dustin 3 years ago.
buildsteps can trigger schedulers
bb-item56-0.7.6fix.patch Download (0.5 KB) - added by gward 2 years ago.
0.7.6 compatibility fix (allows passing Trigger instances to factory.addStep())
triggerable.patch Download (13.9 KB) - added by dustin 2 years ago.
customprops_trigger.diff Download (2.9 KB) - added by redsymbol 2 years ago.
Custom props to triggers for 0.7.6. Assumes customprops076.tgz and customprops_web.diff from ticket #87 have been applied, as well as trigger.patch and bb-item56-0.7.6fix.patch from this ticket

Change History

Changed 3 years ago by dustin

buildsteps can trigger schedulers

Changed 3 years ago by dustin

Note that the tests in this patch require the patch from #57.

Changed 3 years ago by dustin

  • version 0.7.5 deleted

Also note that, if #71 is applied, this patch should be modified to use it.

Changed 2 years ago by warner

  • milestone set to 0.8.0

Changed 2 years ago by gward

This patch is still useful and relevant with Buildbot 0.7.6. It applies cleanly, and *almost* works. In particular, it works fine as long as you keep using the 0.7.5 style of

factory.addStep(Trigger, ...)

but if you switch to the 0.7.6 style

factory.addStep(Trigger(...))

it fails. This is because BuildStep.init() is not saving enough kwargs in self.factory, which in turn is because Trigger.init() does not pass all of its kwargs up. I'll attach one possible fix (as a patch relative to Dustin's original patch).

Changed 2 years ago by gward

0.7.6 compatibility fix (allows passing Trigger instances to factory.addStep())

Changed 2 years ago by gward

  • cc gerg.ward+buildbot@… added

Changed 2 years ago by dustin

Changed 2 years ago by dustin

  • milestone changed from 0.8.0 to 0.7.7

Updated patch against latest head. This, too,has been requested a few times on the mailing list, so I'd like to see it in the next release. Available as 'triggerable.patch' in  http://darcs.r.igoro.us/buildbot/trunk.

Changed 2 years ago by spiff

Is this patch missing a patch to sourcestamp.py? I don't see where getAbsoluteSourceStamp is defined.

Changed 2 years ago by dustin

spiff -- that's in #71.

Changed 2 years ago by redsymbol

Custom props to triggers for 0.7.6. Assumes customprops076.tgz and customprops_web.diff from ticket #87 have been applied, as well as trigger.patch and bb-item56-0.7.6fix.patch from this ticket

Changed 2 years ago by dustin

everything but the customprops patch is available as "#56:trigger.patch" from  http://darcs.r.igoro.us/buildbot/dustin/.

Changed 2 years ago by bhearsum

  • cc bhearsum@… added

Changed 2 years ago by warner

I've applied all the 56-related patches from dustin's repository, and done some further cleanups on top of them (renamed schedulers= to schedulerNames=, and make updateSourceStamp be True by default). I'll look at the "custom props" patches now.

Changed 2 years ago by warner

  • summary changed from triggerable schedulers to custom properties for triggerable schedulers
  • milestone changed from 0.7.7 to 0.7.8

Ok, I don't understand the "custom properties" stuff, like what is the difference between a custom property and a regular one. I'm going to rename the remainder of this ticket to "custom properties for triggerable schedulers" and push it out to the next release. At least the base triggerable stuff is now in 0.7.7 .

Changed 23 months ago by dustin

  • status changed from new to closed
  • resolution set to fixed

This got solved in #87/#124.

Note: See TracTickets for help on using tickets.