TestEngineering/Performance/Sheriffing/Workflow: Difference between revisions

→‎Alert caused by a backout: removed section because we added the details elsewhere
(→‎Alert caused by a backout: removed section because we added the details elsewhere)
 
Line 442: Line 442:
# '''''decision''''' alerts usually are just spikes and not relevant, which is why we are setting them as ''invalid''. In the case that, by looking at the coresponding graph, the regression wasn't improved / fixed, then check with the other sheriffs in the Sheriffs of Performance channel on Element since it might be a cause of concern.
# '''''decision''''' alerts usually are just spikes and not relevant, which is why we are setting them as ''invalid''. In the case that, by looking at the coresponding graph, the regression wasn't improved / fixed, then check with the other sheriffs in the Sheriffs of Performance channel on Element since it might be a cause of concern.
# '''''instrumented''''' in the Tags & Options column should be set as ''invalid''. Alerting will be deactivated for these tests in the future.
# '''''instrumented''''' in the Tags & Options column should be set as ''invalid''. Alerting will be deactivated for these tests in the future.
== Alert caused by a backout ==
In this case we reply to the backout comment with the summary of the alert and adding the ''perf-alert'' keyword.


== Regression which was already backedout ==
== Regression which was already backedout ==
15

edits