Use the last try: in a push's commit messages

RESOLVED FIXED

Status

Release Engineering
Other
P2
normal
RESOLVED FIXED
7 years ago
4 years ago

People

(Reporter: lsblakk, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [try-server])

Attachments

(1 attachment)

Created attachment 479472 [details] [diff] [review]
reversed(changes) to get latest commit message's try: first

This came up in bug 600474 where each commit in a push had a try: line in it and the oldest one's -t none was the one used instead of the last commit's -t all

Reversing the changes when looking through the json push data gives the latest try: syntax to the scheduler.
Attachment #479472 - Flags: review?(catlee)
Duplicate of this bug: 600631
(Reporter)

Comment 2

7 years ago
Tested in staging with:


changeset:   53752:98e9581c7b83
user:        Lukas Blakk <lsblakk@mozilla.com>
date:        Wed Sep 29 11:44:42 2010 -0700
summary:     commit 1 try: -b o -p linux -m none -u none -t all

changeset:   53753:f8846de82ccb
user:        Lukas Blakk <lsblakk@mozilla.com>
date:        Wed Sep 29 11:44:59 2010 -0700
summary:     commit 2 try: -b o -p linux64 -m none -u none -t all

changeset:   53754:241ab9d7c1b7
tag:         tip
user:        Lukas Blakk <lsblakk@mozilla.com>
date:        Wed Sep 29 11:45:16 2010 -0700
summary:     commit 3 try: -b o -p macosx64 -m none -u none -t all

And got only the opt macosx64 build

Updated

7 years ago
Attachment #479472 - Flags: review?(catlee) → review+
(Reporter)

Updated

7 years ago
Flags: needs-reconfig?
Landing on Monday.
Flags: needs-reconfig? → needs-reconfig+
Landed.
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Comment on attachment 479472 [details] [diff] [review]
reversed(changes) to get latest commit message's try: first

717110f59d82
Attachment #479472 - Flags: checked-in+
(Assignee)

Updated

4 years ago
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.