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

Clarify inhibition heuristics with equal labels #1269

Merged
merged 1 commit into from
Jan 29, 2019

Conversation

tbregolin
Copy link
Contributor

@fabxc @brian-brazil This clarifies the intended behaviour of inhibitions in regards to equal labels, as discussed at prometheus/alertmanager#1727 and prometheus/alertmanager#507

@@ -196,6 +196,10 @@ when an alert (source) exists that matches another set of matchers.
Both target and source alerts must have the same label values
for the label names in the `equal` list.

Prometheus interprets any missing label as a label with an empty value.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's not an interpretation, they're the same thing semantically.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated. Thanks.

Prometheus interprets any missing label as a label with an empty value.
Therefore, if all the label names listed in `equal` are missing from both the
source and target alerts, the inhibition rule will apply.

__Alerts can inhibit themselves. Avoid writing inhibition rules where
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is out of date

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! Removed, was fixed at prometheus/alertmanager#1017.

Signed-off-by: Thomás S. Bregolin <thoms3rd@gmail.com>
@tbregolin tbregolin force-pushed the 1727-am-inhibition-docs branch from 675d0ec to 2222d37 Compare January 28, 2019 21:34
@brian-brazil brian-brazil merged commit 74ae0da into prometheus:master Jan 29, 2019
@brian-brazil
Copy link
Contributor

Thanks!

@tbregolin tbregolin deleted the 1727-am-inhibition-docs branch January 29, 2019 10:49
aylei pushed a commit to aylei/docs that referenced this pull request Oct 28, 2019
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