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 severity label to support PD dynamic notifications #98

Closed
peppepetra opened this issue Nov 15, 2023 · 1 comment
Closed

Add severity label to support PD dynamic notifications #98

peppepetra opened this issue Nov 15, 2023 · 1 comment

Comments

@peppepetra
Copy link

Enhancement Proposal

All alerts coming from nrpe-exporter have an hardcoded severity set to critical.

In order to support PagerDuty dynamic-notifications would be beneficial if the status of the check is reflected into the severity label of the alert.

An nrpe check can have one of the following status code that needs to be translate to a severity label as following

  • 0 = OK -> this doesn't produce any alert

  • 1 = Warning -> severity=warning

  • 2 = Critical -> severity=critical

  • 3 = Unknown -> severity=error

@lucabello
Copy link
Contributor

This has been resolved by #101

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants