try server talos builds no longer labeled with pusher and patch name

RESOLVED FIXED

Status

Release Engineering
General
P2
normal
RESOLVED FIXED
9 years ago
5 years ago

People

(Reporter: dbaron, Assigned: bhearsum)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

9 years ago
On http://tinderbox.mozilla.org/MozillaTry/ the talos builds used to have the pusher (e.g., "dbaron@mozilla.com") and patch name printed in the cell on tinderbox, but they no longer do.  I first noticed this a few days ago, so I think it regressed in the past week or two.

Expected results: Every build cell on http://tinderbox.mozilla.org/MozillaTry/ should have the LDAP account name of the person who caused the builds, and the patch name they gave

Actual results: the talos builds don't.
(Assignee)

Comment 1

9 years ago
I'll have a look at this tomorrow
Assignee: nobody → bhearsum
Status: NEW → ASSIGNED
Priority: -- → P2
(Assignee)

Comment 2

9 years ago
Created attachment 378606 [details] [diff] [review]
properly parse the filename/fileURL

My Try Server cleanup ended up with the builds being named a little differently - this patch fixes Talos to be aware of it. the for loop is to support both dir formats (one is used by CVS builds, one is used by Mercurial builds).
Attachment #378606 - Flags: review?(catlee)

Updated

9 years ago
Attachment #378606 - Flags: review?(catlee) → review+
(Assignee)

Updated

9 years ago
Attachment #378606 - Flags: checked‑in+
(Assignee)

Comment 3

9 years ago
Comment on attachment 378606 [details] [diff] [review]
properly parse the filename/fileURL

changeset:   1155:73c590802b57
(Assignee)

Comment 4

9 years ago
Things look OK in the latest builds.
Status: ASSIGNED → RESOLVED
Last Resolved: 9 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.