Closed
Bug 909816
Opened 12 years ago
Closed 12 years ago
[graph server] Remove obsolete ignore_tests option from analyze_talos.py
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: mbrubeck, Assigned: mbrubeck)
References
Details
Attachments
(1 file)
|
2.37 KB,
patch
|
mbrubeck
:
review-
|
Details | Diff | Splinter Review |
The "ignore_tests" option was added in http://hg.mozilla.org/graphs/rev/eaafce2d1a8e and used in bug 535023 to ignore "Nokia" tests. Since we don't run those tests anymore, it currently has no effect. This patch removes it.
Bug 904322 added a new "suppress_email_subjects" option that can be used for the same purpose, but is more versatile because it can match on any part of the subject including the platform name. (The old option could only match the test name, so it wasn't useful for bug 904322.)
Attachment #796096 -
Flags: review?(catlee)
|
||
Comment 1•12 years ago
|
||
ignore_tests actually prevented processing of certain test suites, not only suppressing the reporting of them.
currently I have "ignore_tests = Codesighs" in production. do we still need that?
|
Assignee | |
Comment 2•12 years ago
|
||
Comment on attachment 796096 [details] [diff] [review]
patch
Okay, let's just WONTFIX this bug if the option is still useful.
Attachment #796096 -
Flags: review?(catlee) → review-
|
|
Assignee | |
Updated•12 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → WONTFIX
|
||
Updated•8 years ago
|
Component: Tools → General
You need to log in
before you can comment on or make changes to this bug.
Description
•