490581 - All builders should TinderboxPrint their names

Status: RESOLVED FIXED

(Mozilla Messaging Graveyard :: Release Engineering, defect)

Description

[Philippe M. Chiasson (:gozer)]

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

Comment 1

[Philippe M. Chiasson (:gozer)]

Attachment: [checked in] Tinderbox the slavename

Comment 2

[[e-mail] (:bhearsum)]

Comment on attachment 377718 [details] [diff] [review]
[checked in] Tinderbox the slavename

Yeah, this seems like a good idea.

Comment 3

[[e-mail] (:bhearsum)]

Comment on attachment 377718 [details] [diff] [review]
[checked in] Tinderbox the slavename

changeset:   294:e37c0b140d3a

Comment 4

[Robert Kaiser]

Attachment: [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.

Comment 5

[[e-mail] (:bhearsum)]

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

thanks for catching this KaiRo

Comment 6

[Robert Kaiser]

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 :)

Comment 7

[Philippe M. Chiasson (:gozer)]

(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!