All builders should TinderboxPrint their names

RESOLVED FIXED

Status

Mozilla Messaging
Release Engineering
RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: gozer, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

9 years ago
When a single host is busted, it would make it very easy to spot if it always TinderboxPrint'ed it's hostname, instead of just logging it. Something like

host: momo-vm-xx
(Reporter)

Comment 1

9 years ago
Created attachment 377718 [details] [diff] [review]
[checked in] Tinderbox the slavename
Attachment #377718 - Flags: review?(bhearsum)
Comment on attachment 377718 [details] [diff] [review]
[checked in] Tinderbox the slavename

Yeah, this seems like a good idea.
Attachment #377718 - Flags: review?(bhearsum) → review+
Comment on attachment 377718 [details] [diff] [review]
[checked in] Tinderbox the slavename

changeset:   294:e37c0b140d3a
Attachment #377718 - Attachment description: Tinderbox the slavename → [checked in] Tinderbox the slavename

Comment 4

9 years ago
Created attachment 378764 [details] [diff] [review]
[checked in] make it work on Windows as well

I noticed that it doesn't work on Windows because we end up with quotes around the whole line. This patch at least reduces the quoted part to be what's after the TinderboxPrint: so that it actually is displayed on Windows columns as well.
Attachment #378764 - Flags: review?(bhearsum)
Comment on attachment 378764 [details] [diff] [review]
[checked in] make it work on Windows as well

thanks for catching this KaiRo
Attachment #378764 - Flags: review?(bhearsum) → review+

Comment 6

9 years ago
Comment on attachment 378764 [details] [diff] [review]
[checked in] make it work on Windows as well

Pushed as http://hg.mozilla.org/build/buildbotcustom/rev/b9f1bae782f4 - I'll leave it up to gozer to mark the bug fixed :)
Attachment #378764 - Attachment description: make it work on Windows as well → [checked in] make it work on Windows as well
(Reporter)

Comment 7

9 years ago
(In reply to comment #6)
> (From update of attachment 378764 [details] [diff] [review])
> Pushed as http://hg.mozilla.org/build/buildbotcustom/rev/b9f1bae782f4 - I'll
> leave it up to gozer to mark the bug fixed :)

Thanks for the troubleshooting KaiRo. Works great!
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.