Skip to content
This repository was archived by the owner on Aug 9, 2021. It is now read-only.

alerting rules file contains wrong expressions #80

Closed
p-se opened this issue Oct 14, 2019 · 0 comments · Fixed by #81
Closed

alerting rules file contains wrong expressions #80

p-se opened this issue Oct 14, 2019 · 0 comments · Fixed by #81

Comments

@p-se
Copy link
Collaborator

p-se commented Oct 14, 2019 •
edited
Loading

Rules like node_load1 > 1 and node_load1 > 3 in alert.rules don't make much sense and should likely be changed to either contain node_load1 > 3 or node_load1 < 3 and node_load1 > 1.

https://github.com/ricardoasmarques/ceph-dev-docker/blob/master/prometheus/alert.rules#L16
https://github.com/ricardoasmarques/ceph-dev-docker/blob/master/prometheus/alert.rules#L26
https://github.com/ricardoasmarques/ceph-dev-docker/blob/master/prometheus/alert.rules#L36
https://github.com/ricardoasmarques/ceph-dev-docker/blob/master/prometheus/alert.rules#L46
https://github.com/ricardoasmarques/ceph-dev-docker/blob/master/prometheus/alert.rules#L56
Devp00l pushed a commit to Devp00l/ceph-dev-docker that referenced this issue Oct 14, 2019
Fixes prometheus rule expressions
be64ffb 
Fixes: ricardoasmarques#80
Signed-off-by: Stephan Müller <smueller@suse.com>
@Devp00l Devp00l mentioned this issue Oct 14, 2019
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes prometheus rule expressions Devp00l/ceph-dev-docker
1 participant
@p-se