consolidated unittest builders not using unittest errorparser?

VERIFIED FIXED

Status

--
major
VERIFIED FIXED
10 years ago
5 years ago

People

(Reporter: ted, Assigned: catlee)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

It appears that the new consolidated unittest builders may not be specifying the unittest errorparser in their TinderboxMailNotifier. See the URL for an example of bad error parsing. Should be a simple fix, but it makes it really hard to use these machines for production at the moment.

Updated

10 years ago
Assignee: nobody → catlee
(Assignee)

Comment 1

10 years ago
Created attachment 354184 [details] [diff] [review]
Use unittest errorparser for unit test builders

Works on staging.
Attachment #354184 - Flags: review?(bhearsum)

Updated

10 years ago
Attachment #354184 - Flags: review?(bhearsum) → review+
Comment on attachment 354184 [details] [diff] [review]
Use unittest errorparser for unit test builders

changeset:   604:b183081c67bf
Attachment #354184 - Flags: checked‑in+ checked‑in+
(Assignee)

Comment 3

10 years ago
Now running in production.

All unit tests that start after 9:50am PST today should go through the unittest errorparser.
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
V.Fixed, per, for example,

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox3.1/1229970377.1229974556.8842.gz
OS X 10.5.2 mozilla-1.9.1 unit test on 2008/12/22 10:26:17

and most visibly per
Bad
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1229962243.1229965250.2521.gz
Linux mozilla-central unit test on 2008/12/22 08:10:43
Good
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1229965730.1229970181.25448.gz
Linux mozilla-central unit test on 2008/12/22 09:08:50
Severity: normal → major
Status: RESOLVED → VERIFIED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.