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

Add Alertmanager cases and transposer #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mvxt
Copy link
Collaborator

@mvxt mvxt commented Dec 13, 2024

Summary

  • If there are no alerts in the array of alerts from Alertmanager:
    • Summary will say "Alert from {receiver name}" instead of just "{receiver name}"
    • Description/Body will say "{receiver name} sent a webhook without any alerts or other information"
  • If there is only one alert in the array:
    • Summary of FireHydrant alert will be the summary of the one alert (unchanged from CEL transposer)
    • Body of FireHydrant alert will be "{alert summary}: {alert description}\n"
  • If there are multiple alerts in the array:
    • Summary will be "{number} alerts from {receiver name}"
    • Body of FireHydrant alert will be "{alert summary}: {alert description}\n" for every alert in array

Rather than summarizing multiple alerts only by the first alert's title and description (current behavior), we are including all alert summaries and bodies. The intention is to provide more clarity for multi-alert scenarios.

Annotations, links, and other param behaviors remain unchanged from CEL transposer.

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.

1 participant