Opened 13 years ago

Closed 12 years ago

#124 closed enhancement (duplicate)

WIthProperties uses properties from builders, schedulers and slaves

Reported by: hollowman Owned by:
Priority: major Milestone: 0.7.8
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 (5)

comment:1 Changed 12 years ago 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.

comment:2 Changed 12 years ago by gward

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

comment:3 Changed 12 years ago by dustin

  • Cc Pike dossy dustin gward added

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.

comment:4 Changed 12 years ago 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.

comment:5 Changed 12 years ago by dustin

  • Resolution set to duplicate
  • Status changed from new to closed

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.

Note: See TracTickets for help on using tickets.