Closed
Bug 1030428
Opened 9 years ago
Closed 9 years ago
Log parsing regex is missing failures compared to TBPL
Categories
(Tree Management :: Treeherder, defect, P1)
Tree Management
Treeherder
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 1023868
People
(Reporter: automatedtester, Assigned: camd)
References
Details
Attachments
(2 files)
No description provided.
Reporter | ||
Comment 1•9 years ago
|
||
how TBPL renders the same text
Reporter | ||
Comment 2•9 years ago
|
||
the extra data shown makes it simple to star when there are multiple people looking at the trees
Comment 3•9 years ago
|
||
The TBPL failure excerpt only shows lines that match the regex, rather than the N lines leading up to it, the issue in the screenshot is actually due to two problems: 1) The log line order is inverted in treeherder (we're showing new-to-old rather than old-to-new). 2) The treeherder regex isn't quite complete (albeit easily fixed :-)). I'll make this about #2 and file another for #1, since I raised this at a treeherder meeting a few weeks ago, must have not made it to an etherpad, since we talked about so many things during the meeting.
OS: Mac OS X → All
Hardware: x86 → All
Summary: failure details doesnt show the last x items before the error like tbpl → Log parsing regex is missing failures compared to TBPL
Comment 4•9 years ago
|
||
(In reply to Ed Morley [:edmorley UTC+0] from comment #3) > 1) The log line order is inverted in treeherder (we're showing new-to-old > rather than old-to-new). Bug 1030619
Comment 5•9 years ago
|
||
Actually, this might just turn out to be bug 1030620 (the regex do need a bit of work, but I don't think it was them in this case)
Updated•9 years ago
|
Blocks: treeherder-sheriff-transition
Updated•9 years ago
|
Severity: normal → critical
Updated•9 years ago
|
Priority: -- → P1
Assignee | ||
Comment 6•9 years ago
|
||
We have PROCESS-CRASH as a regex to be found, so I'll double-check that we have that covered in a unit test, so it shows in the log artifact.
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → cdawson
Status: NEW → ASSIGNED
Assignee | ||
Comment 7•9 years ago
|
||
Hmm, I see other jobs that DO get the PROCESS-CRASH error line to show. And we have specific unit tests for this error case which pass. I have a suspicion it may be caused by bug 1023868. Can't confirm that yet, but I'm going to search for another case of this...
Assignee | ||
Comment 8•9 years ago
|
||
We are picking up PROCESS-CRASH failure lines as seen here: https://treeherder.mozilla.org/ui/logviewer.html#?job_id=87011&repo=mozilla-central It is my belief that we missed this because we were missing >1hr steps. So I'm going to close this as a dup now. Please reopen if you see it.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•