Closed Bug 723508 Opened 12 years ago Closed 12 years ago

Uptake monitoring shouldn't start before updates are ready

Categories

(Release Engineering :: General, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: rail, Assigned: rail)

References

Details

(Whiteboard: [releases][automation])

Attachments

(1 file, 1 obsolete file)

It's OK to push files to the mirrors in post 11.0b1 before updates are ready. However, uptake monitoring should start when both updates and push to mirrors are happened, to prevent "ready for releasetest testing" and "ready for release" emails.
Attached patch builbotcustom (obsolete) — Splinter Review
* add almost dummy builders to trigger uptake monitoring (instead of triggering them right away after push to mirrors)
* add another AggregatingScheduler to start uptake monitoring when both updates and push to mirrors are happened
Attachment #594170 - Flags: review?(bhearsum)
Comment on attachment 594170 [details] [diff] [review]
builbotcustom

Review of attachment 594170 [details] [diff] [review]:
-----------------------------------------------------------------

::: process/release.py
@@ +1083,5 @@
>  
> +    if not releaseConfig.get('disableBouncerEntries'):
> +        trigger_uptake_factory = BuildFactory()
> +        trigger_uptake_factory.addStep(Trigger(
> +            schedulerNames=[builderPrefix('ready-for-rel-test'),

Can we make the trigger of the ready-for-rel-test builder conditional on having verifyConfigs, like we do for other updates things? It's unnecessary noise if there's no updates.
Attachment #594170 - Flags: review?(bhearsum) → review+
Attached patch buildbotcustomSplinter Review
interdiff: https://gist.github.com/1730935
Attachment #594170 - Attachment is obsolete: true
Attachment #594192 - Flags: review?(bhearsum)
Attachment #594192 - Flags: review?(bhearsum) → review+
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: