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

Avoid pagerduty alerts duplication #61

Closed
helgharbi opened this issue Nov 9, 2023 · 2 comments · Fixed by #76
Closed

Avoid pagerduty alerts duplication #61

helgharbi opened this issue Nov 9, 2023 · 2 comments · Fixed by #76

Comments

@helgharbi
Copy link

helgharbi commented Nov 9, 2023

Enhancement Proposal

When using pagerduty_configs with alert manager alertmanager.

Raised pagerduty alerts are not using dedupication key to correlating triggers and resolves.
Causing a new alerts to be trigered every interval and alerts are never resolved.

See dedup_key usage here:

dedup_key (String): Deduplication key for correlating triggers and resolves

@sed-i
Copy link
Contributor

sed-i commented Dec 14, 2023

In order to avoid duplicating alerts, the repeat interval key needs to be set to a large value to suppress additional messages.
As for dedup_key and resolution - these could be raised as a feature request for cos-alerter to add native pagerduty support.

@simskij
Copy link
Member

simskij commented May 21, 2024

In order to avoid duplicating alerts, the repeat interval key needs to be set to a large value to suppress additional messages.

This is not a viable solution.

As for dedup_key and resolution - these could be raised as a feature request for cos-alerter to add native pagerduty support.

However, this is :)

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

Successfully merging a pull request may close this issue.

3 participants