Closed
Bug 1643685
Opened 5 years ago
Closed 5 years ago
Perfherder applink data is using the default "browsertime" test name
Categories
(Testing :: mozperftest, defect, P1)
Testing
mozperftest
Tracking
(firefox-esr68 unaffected, firefox77 unaffected, firefox78 unaffected, firefox79 fixed)
RESOLVED
FIXED
mozilla79
| Tracking | Status | |
|---|---|---|
| firefox-esr68 | --- | unaffected |
| firefox77 | --- | unaffected |
| firefox78 | --- | unaffected |
| firefox79 | --- | fixed |
People
(Reporter: sparky, Assigned: sparky)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
Currently the test names in app-link perftest data in CI is "browsertime" which is the default value - it should be "App-link".
|
Assignee | |
Comment 1•5 years ago
|
||
This patch fixes the test name being used for the App-link test in CI (regression). The patch also adds a new feature to ScriptInfo which checks to make sure that the test doesn't have any extra unknown keys in it. If it does, a failure is raised. This change is to attempt to help prevent these kinds of regressions from happening again. A new test is added for this functinality as well.
|
||
Updated•5 years ago
|
Keywords: regression
|
|
||
Comment 2•5 years ago
|
||
Set release status flags based on info from the regressing bug 1641834
status-firefox77:
--- → unaffected
status-firefox78:
--- → unaffected
status-firefox79:
--- → affected
status-firefox-esr68:
--- → unaffected
Pushed by gmierz2@outlook.com:
https://hg.mozilla.org/integration/autoland/rev/bac92eaf4731
Use name instead of test_name when finding the metric test name. r=mozperftest-reviewers,tarek
|
||
Comment 4•5 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla79
You need to log in
before you can comment on or make changes to this bug.
Description
•