794895 - Replace Talos "FAIL:" with "talosError:" rather than "TinderboxPrint:FAIL:", so they appear in TBPL annotated summary

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[Ed Morley [:emorley]]

Talos outputs failures with the prefix "FAIL:".

buildbotcustom's talos.py then replaces "FAIL:" with "TinderboxPrint:FAIL:", so that TBPL's tinderbox_print parser puts them in the panel lower right of the TBPL UI.

However now that we have bug 790602, we should just get talos.py to prefix with "talosError:" instead, so that TBPL displays them in the annotated summary, rather than hiding them away in the lower tinderbox_print panel.

Depends on bug 794799, since before that lands, the "FAIL:" output will be too noisy, and will swamp the annotated summary box.

Comment 1

[Ed Morley [:emorley]]

Attachment: Patch v1

Comment 2

[Ed Morley [:emorley]]

Comment on attachment 665379 [details] [diff] [review]
Patch v1

https://hg.mozilla.org/build/buildbotcustom/rev/dcb42bc70954

Comment 3

[Aki Sasaki (not active)]

This is now live.

Comment 4

[Ed Morley [:emorley]]

Thank you :-)