The bump of the version on default for a release triggers dependent builds

RESOLVED FIXED

Status

Release Engineering
General
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: armenzg, Assigned: bhearsum)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

When the bump lands [1] on default it triggers a bunch of dependent builds [2]

The commit message looks like this:
"Automated checkin: version bump for firefox 3.6.26 release. CLOSED TREE a=release"

Adding DONTBUILD should do the trick to prevent this misuse of resources.

This sounds like a good intern bug :)

[1] http://hg.mozilla.org/releases/mozilla-1.9.2/rev/0219ac257e3c
[2] https://tbpl.mozilla.org/?tree=Firefox3.6&rev=0219ac257e3c
(In reply to Armen Zambrano G. [:armenzg] - Release Engineer from comment #1)
> For the record this does not seem to be happening for Mozilla-Beta.
> 
> https://tbpl.mozilla.org/?tree=Mozilla-Beta&rev=5b94a143a4ec
> https://tbpl.mozilla.org/?tree=Mozilla-Beta&rev=f2a16e2a67ca

Thats because on mozilla-beta we don't do a change on |default| we relbranch and do our (fake) version update on that. 3.6.x has the unique (now) situation of needing to change the default version #
(Assignee)

Comment 3

6 years ago
We should just add DONTBUILD to the commit message.
Assignee: nobody → bhearsum
(Assignee)

Comment 4

6 years ago
Created attachment 596557 [details] [diff] [review]
add DONTBUILD to tag and bump commits

I don't think there's any reason that these should ever be non-DONTBUILD.
Attachment #596557 - Flags: review?(rail)
Comment on attachment 596557 [details] [diff] [review]
add DONTBUILD to tag and bump commits

The logic looks good, but the patch needs to be refreshed. The current one failed to apply.
Attachment #596557 - Flags: review?(rail) → review-
(Assignee)

Comment 6

6 years ago
Created attachment 596986 [details] [diff] [review]
unbitrotted

I also removed an unused import.
Attachment #596557 - Attachment is obsolete: true
Attachment #596986 - Flags: review?(rail)
Attachment #596986 - Flags: review?(rail) → review+
(Assignee)

Comment 7

6 years ago
Comment on attachment 596986 [details] [diff] [review]
unbitrotted

Landed: changeset:   2234:f16684d20b34
Attachment #596986 - Flags: checked-in+
(Assignee)

Updated

6 years ago
Status: NEW → RESOLVED
Last Resolved: 6 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.