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 excludes in config lelvel, not only alerts yaml. #48

Closed
Veres72 opened this issue Sep 20, 2021 · 1 comment · Fixed by #81
Closed

Add excludes in config lelvel, not only alerts yaml. #48

Veres72 opened this issue Sep 20, 2021 · 1 comment · Fixed by #81
Assignees

Comments

@Veres72
Copy link

Veres72 commented Sep 20, 2021

Hello.
Could you add ignoring alerts by name or regexp via config file?
It will be nice for cases when there is no way to declare excludes in alerts declaration.
E.g we want to lint alerts in multitenancy cluster and send a notification to teams, which have alerts without time series. Teams are deploying their alerts as promerheusRules object and there is not possible to keep comments.

@prymitive prymitive self-assigned this Sep 30, 2021
prymitive added a commit that referenced this issue Nov 26, 2021
prymitive added a commit that referenced this issue Nov 26, 2021
@prymitive
Copy link
Collaborator

Added ignore blocks to match - #81.
Plus there's a name field on match and ignore to allow matching by alert and record fields on prometheus rules.
Example:

rule {
    ignore {
        name = "Ignore .+"
    }
    ...
}

Would ignore this alert:

- alert: Ignore This Alert
  expr: sum(foo) > 0

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 a pull request may close this issue.

2 participants