Closed Bug 713575 Opened 9 years ago Closed 9 years ago

Brief logs 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)

If someone gives you a link to a brief log, your only options to see what slave it ran on (when the answer is "it failed like that because the slave is broken") are to load the full log or download the raw log. Given the ridiculous size of some of our logs, that's a lot to get at ten or fifteen characters. We should instead just stick it in the brief log, either as the first line in the "Summary" section or at the end of the "platform test on date for push" line.
Attached patch patchSplinter Review
Simple but untested, because I haven't set up a working server deployment yet.
Assignee: nobody → mbrubeck
Status: NEW → ASSIGNED
Attachment #609799 - Flags: review?
Attachment #609799 - Flags: review? → review?(mstange)
Comment on attachment 609799 [details] [diff] [review]
patch

Review of attachment 609799 [details] [diff] [review]:
-----------------------------------------------------------------

You might want to add that to FullLogGenerator as well.
(In reply to Arpad Borsos (Swatinem) from comment #2)
> You might want to add that to FullLogGenerator as well.

The slave name is already included right at the start of the full log, so I don't think this would be very useful (unless maybe people are writing scrapers or custom userstyles/userscripts that would consume this data somehow).
Attachment #609799 - Flags: review?(mstange) → review+
https://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/f9d7e61f6d8a
Status: ASSIGNED → RESOLVED
Closed: 9 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.