Closed Bug 493818 Opened 15 years ago Closed 15 years ago

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

Categories

(Release Engineering :: General, defect, P2)

Product:
Release Engineering
Component:
General
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: dbaron, Assigned: bhearsum)

Details

Attachments

(1 file)

properly parse the filename/fileURL
1.13 KB, patch
catlee
: review+
bhearsum
: checked-in+
Details | Diff | Splinter Review 
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.
I'll have a look at this tomorrow
Assignee: nobody → bhearsum
Status: NEW → ASSIGNED
Priority: -- → P2
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)
Attachment #378606 - Flags: review?(catlee) → review+
Attachment #378606 - Flags: checked‑in+
Comment on attachment 378606 [details] [diff] [review]
properly parse the filename/fileURL

changeset:   1155:73c590802b57
Things look OK in the latest builds.
Status: ASSIGNED → RESOLVED
Closed: 15 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

Creator:
Created:
Updated:
Size: