713575 - Brief logs should include the slavename

Status: RESOLVED FIXED

(Tree Management Graveyard :: TBPL, defect)

Description

[Phil Ringnalda (:philor)]

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.

Comment 1

[Matt Brubeck (:mbrubeck)]

Attachment: patch

Simple but untested, because I haven't set up a working server deployment yet.

Comment 2

[Arpad Borsos [:Swatinem]]

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.

Comment 3

[Matt Brubeck (:mbrubeck)]

(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).

Comment 4

[Matt Brubeck (:mbrubeck)]

https://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/f9d7e61f6d8a