Skip to content

feat: add opsgenie priority template based on severity label#3870

Closed
ncrmro wants to merge 1 commit intoprometheus:mainfrom
ncrmro:patch-1
Closed

feat: add opsgenie priority template based on severity label#3870
ncrmro wants to merge 1 commit intoprometheus:mainfrom
ncrmro:patch-1

Conversation

@ncrmro
Copy link

@ncrmro ncrmro commented Jun 6, 2024

P1 in my opinion is reserved for actual outage, in our case if cloudflare fails the external domain health check though I'm open to updating this default.

Signed-off-by: Nicholas Romero <ncrmro@gmail.com>
@ncrmro ncrmro changed the title feat: add opsgenie priorty levels based on severity label feat: add opsgenie priority template based on severity label Jun 6, 2024
@grobinson-grafana
Copy link
Collaborator

Hi! 👋 Thanks for opening this PR. As mentioned here, I think the same applies to priority label:

Alertmanager doesn't have severity, and Prometheus users are not required to set use a severity label, so I'm not sure if this is something that will be accepted. Is it not possible to template the severity for Opsgenie using the existing templating infrastructure?

Why not use a custom template to set the priority field in Opsgenie for your use case?

@ncrmro
Copy link
Author

ncrmro commented Jun 20, 2024

@grobinson-grafana we did just that!

@ncrmro ncrmro closed this Jun 20, 2024
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.

2 participants