Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Alertmanager notification template #1764

Merged
merged 3 commits into from
Dec 17, 2024
Merged

Conversation

TheoBrigitte
Copy link
Member

This PR introduces the following changes into the Alertmanager notification template:

  • Update alert and query URLs for Mimir to point to the Active notification rather than the rule page
  • Move link section (runbook, dashboard, explors) before instance to avoid them being lost due to OpsGenie max description being reached
  • Move the warnings for missing runbook and dashboard up into the link section

- Update alert and query URLs for Mimir to point to the Active notification rather than the rule page
- Move link section (runbook, dashboard, explors) before instance to avoid them being lost due to OpsGenie max description being reached
- Move the warnings for missing runbook and dashboard up into the link section
@TheoBrigitte TheoBrigitte requested a review from a team as a code owner December 2, 2024 10:50
@TheoBrigitte TheoBrigitte self-assigned this Dec 2, 2024
@TheoBrigitte
Copy link
Member Author

Here is what the new Opsgenie alert and Grafana alerting pages looks like

image
Note: links are now before instances and other informations. instances are empty, because that's just a dummy alert I created

image
Note: this page shows actual active notifications, but there might be duplicate due to alerts being sent to both opsgenie and slack for instance, are we fine with that ?

@TheoBrigitte TheoBrigitte merged commit c13636e into main Dec 17, 2024
5 checks passed
@TheoBrigitte TheoBrigitte deleted the improve-alert-template branch December 17, 2024 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants