Closed
Bug 991134
Opened 11 years ago
Closed 11 years ago
Output an additional failure line for reftest/jsreftest loadfailed:null failures to make bug 789751 and bug 720452 TBPL star-able
Categories
(Testing :: Reftest, defect)
Testing
Reftest
Tracking
(firefox30 fixed, firefox31 fixed)
RESOLVED
FIXED
mozilla31
People
(Reporter: emorley, Assigned: emorley)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
(deleted),
patch
|
dbaron
:
review+
|
Details | Diff | Splinter Review |
I feel dirty suggesting we do this, but the lack of action in bug 720452 doesn't leave me with much other choice.
The failure modes there are not TBPL star-able, since the test name varies so much, so I propose we at least as a temporary measure output an additional failure line here:
https://hg.mozilla.org/mozilla-central/diff/5a24cad1dd8a/layout/tools/reftest/reftest.js
...that can generate a bug suggestion.
Assignee | ||
Updated•11 years ago
|
Summary: Output an addition failure line for jsreftest loadfailed:null failures to make bug 720452 TBPL star-able → Output an additional failure line for jsreftest loadfailed:null failures to make bug 720452 TBPL star-able
Assignee | ||
Updated•11 years ago
|
Blocks: 789751
Summary: Output an additional failure line for jsreftest loadfailed:null failures to make bug 720452 TBPL star-able → Output an additional failure line for reftest/jsreftest loadfailed:null failures to make bug 789751 and bug 720452 TBPL star-able
Assignee | ||
Comment 1•11 years ago
|
||
The failure modes in bug 789751 and bug 720452 appear to affect all reftests indiscriminately, and as such the usual TBPL test filename search fails to succeed, resulting in sheriffs having to manually star the failures.
This patch adds an additional log line for these no-reason failures, which intentionally doesn't follow the "three pipe delimeted fields" format, and so hits the TBPL fallback of searching Bugzilla for the entire line.
Now bug 896840 will hopefully add failure reasons to more of the reftest load flow, to the point where we can fix these failures - but for now making them starable would make the sheriffs significantly more happy about the current situation (and also mean more easily filterable bugspam for those two bugs for everyone else too).
Attachment #8407664 -
Flags: review?(dbaron)
Assignee | ||
Comment 2•11 years ago
|
||
(Trailing semi-colon removed locally)
Comment 3•11 years ago
|
||
Comment on attachment 8407664 [details] [diff] [review]
Patch v1
r=dbaron with the double-;; fixed
Attachment #8407664 -
Flags: review?(dbaron) → review+
Assignee | ||
Comment 4•11 years ago
|
||
Thank you :-)
remote: https://hg.mozilla.org/integration/mozilla-inbound/rev/d849022d8633
Comment 5•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
Comment 6•11 years ago
|
||
status-firefox30:
--- → fixed
status-firefox31:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•