Bug 1942867 - Add an alert management system for telemetry alerts. #9015
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
gmierz
requested review from
a team,
beatrice-acasandrei and
esanuandra
as code owners
…g telemetry bugs.
gmierz
force-pushed
the
telemetry-alert-manager-comp
branch
from
44fab00 to
0e26ca1
Compare
|
Here's a sample bug that is filed by this: https://bugzilla.mozilla.org/show_bug.cgi?id=1993145 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This set of patches adds a new alert management system for telemetry alerts. The commits below attempt to split up the system into some logical chunks with newer commits building on previous ones.
Some generic base and utility classes are added directly to the
auto_perf_sheriffingfolder. These are not specific to telemetry alerting and could be used in other performance sheriffing automation.The concrete classes for telemetry alert management are found in the
treeherder/perf/auto_perf_sheriffingfolder. These are then integrated into the telemetry detection code in Sherlock through theTelemetryAlertManagerand run fromTelemetryAlertManager.manage_alerts.The
manage_alertsmethod is defined generically in theAlertManagerclass. It starts by updating the DB with any changes made in telemetry bugs in Bugzilla - this is only for their resolutions at the moment. After this, bugs are filed for the alerts that are generated for any probes that specify a bug should be filed (by setting themonitor.alertfield to True in their probe definition). Once bugs are filed, modifications are made to these bugs and any existing bugs as needed. This currently only modifies thesee_alsofield to associate all bugs filed for the same detection range together - in other words, all the bugs that are part of the samePerformanceTelemetryAlertSummary. At the end of this "bug handling" phase, emails are produced for any alerts that request it (either bugs are produced or emails, but never both to reduce spamming). Finally, it's possible that either the bug modifications or emails fail. In that case, we have a "house keeping" stage where we do retries of the failed alerts on a daily basis.For treeherder-admins, the relevant changes will be in the first commit where I am adding a new env field to capture the
BUG_COMMENTER_API_KEYbeing set locally. This is needed for testing the bug modification aspect of the management system.