Ticket #124 (closed enhancement: duplicate)

Opened 1 year ago

Last modified 7 months ago

WIthProperties uses properties from builders, schedulers and slaves

Reported by: hollowman Assigned to:
Priority: major Milestone: 0.7.8
Component: configuration Version: 0.7.6
Keywords: Cc: Pike, dossy, dustin, gward

Description

In order to customize steps, we should be able to have external properties depending on the builder, schedulers or slaves.

For example, if I want to use the same builder on different platforms, I may wish to use slave dependent properties/options for my steps.

Furthermore, I sometimes might want to do a full build with a full checkout. 

   slave('linux1','pw1',properties={'os':'linux','arch':'x86','p':'32'})
   slave('linux2','pw2',properties={'os':'linux','arch':'ia64','p':'64'})
   slave('aix1','pass',properties={'os':'aix','arch':'ppc','p':'64'})

   scheduler(...,properties={'clean','none'})
   scheduler(...,properties={'clean','libs'})
   scheduler(...,properties={'clean','full'})

    
   addStep(...,["make",WithProperties("%s","clean")])
   addStep(...,["build",WithProperties("%s","os")]

Change History

03/22/08 13:02:08 changed by warner

  • milestone changed from undecided to 0.7.8.

I like it. I'd also like to have the 'buildbot force' command (which doesn't quite exist yet) to let you set properties too.

03/25/08 11:26:56 changed by gward

Doesn't ticket #87 (custom build properties) address this request? (That is, I think this ticket is a duplicate.)

04/03/08 21:19:03 changed by dustin

  • cc set to Pike, dossy, dustin, gward.

gward, yes, it overlaps signficantly with #87. What #87 doesn't include is specification of properties from slaves, schedulers, and steps. Schedulers are set to be added in #87, but let's keep this ticket open for sourcing properties in steps and slaves, as that's somewhat more challenging, and requires some additional thought.

04/06/08 06:24:08 changed by gward

Ahh, got it. I did not fully understand this feature request when I first looked at it. I retract my "isn't this a dupe?" comment.

04/12/08 14:05:28 changed by dustin

  • status changed from new to closed.
  • resolution set to duplicate.

I think I'm going to move that work into #87, though, since there are some loose ends still to tie up on that bug.