Closed Bug 713574 Opened 12 years ago Closed 12 years ago

tbplbot bug comments should include the slavename

Categories

(Tree Management Graveyard :: TBPL, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: philor, Assigned: mbrubeck)

References

Details

Attachments

(1 file)

The classic tinderbox-based comment was

logurl
platform/branch/test/date

s: slavename
TEST-UNEXPECTED-FAIL: foo

which I think reflected that we were getting the slavename for free after having (gotten|put) it in the summary. A more sensible comment would put it either after the platform/etc. line, or at the end of that line as "on tegra-007", but anywhere at all would be better than nowhere, for recognizing that some failure is limited to one or a few slaves.
Attached patch patchSplinter Review
Assignee: nobody → mbrubeck
Status: NEW → ASSIGNED
Attachment #609789 - Flags: review?(arpad.borsos)
Attachment #609789 - Flags: review?(arpad.borsos) → review+
https://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/cfe4c6b7d338
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Depends on: 743595
Product: Webtools → Tree Management
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: