TestEngineering/Performance/Sheriffing: Difference between revisions

Jump to navigation Jump to search
no edit summary
(Added a video for Overview section)
No edit summary
 
Line 14: Line 14:
** [[/Tree_FAQ#Branch_names_and_confusion|branch]]
** [[/Tree_FAQ#Branch_names_and_confusion|branch]]
** % regressed, this is a range of the regressions (not improvements)
** % regressed, this is a range of the regressions (not improvements)
** the [[TestEngineering/Performance/Talos/Tests|tests]] which have regressed
** the [https://firefox-source-docs.mozilla.org/testing/perfdocs/talos.html#test-definitions tests] which have regressed
** the platforms we see this regression on
** the platforms we see this regression on
* date of the suspect revision push
* date of the suspect revision push
Line 22: Line 22:


Below the summary will be a list of alerts, each alert will reference:
Below the summary will be a list of alerts, each alert will reference:
* [[TestEngineering/Performance/Talos/Tests|Test name]]
* [https://firefox-source-docs.mozilla.org/testing/perfdocs/talos.html#test-definitions Test name]
* platform (including build type, such as opt, pgo)
* platform (including build type, such as opt, pgo)
* old score (median score of the previous 12 commits)
* old score (median score of the previous 12 commits)
Line 101: Line 101:
* Product/Component - this should be the same as the bug which is the root cause, if >1 bug, file in [https://bugzilla.mozilla.org/enter_bug.cgi?product=Testing&component=Talos Talos]
* Product/Component - this should be the same as the bug which is the root cause, if >1 bug, file in [https://bugzilla.mozilla.org/enter_bug.cgi?product=Testing&component=Talos Talos]
* Dependent/Block bugs - For a new bug, add the [[#Tracking_bugs|tracking bug]] (for the current version) and root cause bug(s) as blocking this bug
* Dependent/Block bugs - For a new bug, add the [[#Tracking_bugs|tracking bug]] (for the current version) and root cause bug(s) as blocking this bug
* CC list - cc patch author(s), reviewer(s) and owner of the tests as documented on the [[TestEngineering/Performance/Talos/Tests|Talos tests wiki]]; if we have >1 bug, we should cc everyone who worked on those bugs so we call pitch in an answer questions faster
* CC list - cc patch author(s), reviewer(s) and owner of the tests as documented in the [https://firefox-source-docs.mozilla.org/testing/perfdocs/talos.html#test-definitions Talos test definition]; if we have >1 bug, we should cc everyone who worked on those bugs so we call pitch in an answer questions faster
* Summary of bug should have a check to make sure the revision is accurate
* Summary of bug should have a check to make sure the revision is accurate
* The description is auto suggested as well, please verify the revision here
* The description is auto suggested as well, please verify the revision here
Confirmed users
2,197

edits

Navigation menu