Bug 1728338 - Stop alerting on tests not in alert_on r=perftest-reviewers,sparky

Differential Revision: https://phabricator.services.mozilla.com/D144216
This commit is contained in:
Kash Shampur 2022-04-21 18:08:44 +00:00
parent b9466bda67
commit 8c9b88cd70

View File

@ -1195,6 +1195,14 @@ class RaptorOutput(PerftestOutput):
% measurement_name
)
new_subtest["shouldAlert"] = True
else:
# Explicitly set `shouldAlert` to False so that the measurement
# is not alerted on. Otherwise Perfherder defaults to alerting
LOG.info(
"turning off subtest alerting for measurement type: %s"
% measurement_name
)
new_subtest["shouldAlert"] = False
new_subtest["value"] = filters.median(filtered_values)
@ -1584,6 +1592,14 @@ class BrowsertimeOutput(PerftestOutput):
subtest["shouldAlert"] = True
if self.app in ("chrome", "chrome-m", "chromium"):
subtest["shouldAlert"] = False
else:
# Explicitly set `shouldAlert` to False so that the measurement
# is not alerted on. Otherwise Perfherder defaults to alerting.
LOG.info(
"turning off subtest alerting for measurement type: %s"
% measurement_name
)
subtest["shouldAlert"] = False
subtest["replicates"] = []
suite["subtests"][measurement_name] = subtest
else: