Parse gtest failures from logs

RESOLVED FIXED

Status

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: mt, Assigned: mt)

Tracking

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
I think that it is sufficient to watch for '^\[  FAILED  \]' in logs.  Ideally, this would also walk backwards 4-6 lines until it finds ':\d+: failure$'.

Comment 1

2 years ago
Links to example logs please!
Flags: needinfo?(martin.thomson)
Created attachment 8838309 [details] [review]
[treeherder] martinthomson:bug1340329 > mozilla:master
(Assignee)

Comment 3

2 years ago
Comment on attachment 8838309 [details] [review]
[treeherder] martinthomson:bug1340329 > mozilla:master

Soo fast!

https://public-artifacts.taskcluster.net/rJ9eZDqIQoq46BNVIuPi4Q/0/public/logs/live_backing.log
Flags: needinfo?(martin.thomson)
Attachment #8838309 - Flags: review?(emorley)

Updated

2 years ago
Assignee: nobody → martin.thomson

Comment 4

2 years ago
Comment on attachment 8838309 [details] [review]
[treeherder] martinthomson:bug1340329 > mozilla:master

So in some future world I'd hoped we could significantly reduce the number of cases the log parser has to support, and push that work down to the individual test runners/harnesses etc (for example match only against mozharness style log line error level prefixes, and let mozharness or a taskcluster wrapper decide what counts as an error line).

However given the mixture of job types we have (plus the current lack of standard format for denoting an error line, given that mozharnesses prefixes aren't even used by much of taskcluster), I think that future state is quite far away, so we'll sadly have to accept further regex additions for now.

Thank you for the PR :-)
Attachment #8838309 - Flags: review?(emorley) → review+

Comment 6

2 years ago
This will auto-deploy to Treeherder stage in a few minutes, and will be deployed to Treeherder production at some point soon (typically 1-2 times per week). It's worth noting that the log parser only runs once, so error summaries generated prior to this landing won't be updated.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.