1213317 - Move release promotion buildbot builders to separate module

Status: RESOLVED FIXED

(Release Engineering :: Release Automation, defect)

Description

[Rail Aliiev [:rail]]

It would be great to group all release promotion related builders in a single file module, that can be used from misc.py. http://hg.mozilla.org/build/buildbotcustom/file/ec6cd658f788/misc.py#l3327 is the current entry point.

Comment 1

[Nick Thomas [:nthomas] (UTC+12)]

I think this is a good idea. Originally it went into misc.py because it gave easy access to ci-style objects, since that's what promotion is closer too. I have a vague memory of some sort of circular import problem which that avoided too.

Comment 2

[Jordan Lund (:jlund)]

Attachment: 151017_1213317_give_promotion_logic_its_own_method_entry-buildbotcustom.patch

though not a necessary bug, I took this first as a way to familiarize myself with the buildbot code for build promotion.

this patch:

1) renames generateReleasePromotionObjects to generateReleasePromotionBuilders since we are only creating builder objects and letting bbb schedule/trigger release promotion jobs
2) moves generateReleasePromotionBuilders from misc.py to process/release.py
2) method impl now only returns a list of builders and not a dict of objects and also returns immediately if not a 'release promotion branch' (will make more sense with upcoming buildbot-configs patch

Comment 3

[Jordan Lund (:jlund)]

Attachment: 151017_1213317_give_promotion_logic_its_own_method_entry-buildbot-configs.patch

this patch:

1) rather than calling generateReleasePromotionBuilders from within generateReleaseBranchObjects(), we give release promotion builders their own entry call within master.cfg.
2) since generateReleasePromotionBuilders only tracks new builders, this should only be needed for the production build master.cfg (mozilla/builder_master.cfg) and the staging universal master

a dump master diff yielded a no-op for me locally. hopefully production agrees with that :)

Comment 4

[Jordan Lund (:jlund)]

Attachment: 151017_1213317_give_promotion_logic_its_own_method_entry-buildbot-configs2.patch

same as before but removes old unused var (releasePromotionBuilders) from earlier WIP

interdiff fwiw: http://people.mozilla.org/~jlund/151017_1213317_bbot-cfgs-interdiff.patch

Comment 5

[Rail Aliiev [:rail]]

Hmm, for some reason it doesn't pass check-masters.sh in my environment.

Comment 6

[Rail Aliiev [:rail]]

"my environment" was key here, disregard.

Comment 7

[Jordan Lund (:jlund)]

Comment on attachment 8675342 [details] [diff] [review]
151017_1213317_give_promotion_logic_its_own_method_entry-buildbotcustom.patch

thanks https://hg.mozilla.org/build/buildbotcustom/rev/c91ff584f674

Comment 8

[Jordan Lund (:jlund)]

Comment on attachment 8675345 [details] [diff] [review]
151017_1213317_give_promotion_logic_its_own_method_entry-buildbot-configs2.patch

https://hg.mozilla.org/build/buildbot-configs/rev/3fd8f683b28d

Comment 9

[Rail Aliiev [:rail]]

I think we can close this bug