Skip to content

Rule group metrics are always zero in Cortex #1557

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

Closed
bboreham opened this issue Aug 6, 2019 · 5 comments
Closed

Rule group metrics are always zero in Cortex #1557

bboreham opened this issue Aug 6, 2019 · 5 comments

Comments

@bboreham
Copy link
Contributor

bboreham commented Aug 6, 2019

#1548 prompted me to look at these metrics. Most of what I looked at was zero:

prometheus_rule_group_duration_seconds (this one is also zero in our Prometheus)
prometheus_rule_group_duration_seconds_count
prometheus_rule_group_duration_seconds_sum
prometheus_rule_group_last_duration_seconds
prometheus_rule_group_last_evaluation_timestamp_seconds

In fact the only one which isn't all zeros in Cortex is prometheus_rule_group_rules

@jtlisi
Copy link
Contributor

jtlisi commented Aug 6, 2019

The metrics that should work with the way we utilize the rule group struct are

prometheus_rule_evaluation_duration_seconds
prometheus_rule_evaluation_failures_total
prometheus_rule_group_rules

I think the ones you listed could be added relatively easily with some upstream work

@stale
Copy link

stale bot commented Feb 3, 2020

This issue has been automatically marked as stale because it has not had any activity in the past 30 days. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Feb 3, 2020
@bboreham
Copy link
Contributor Author

bboreham commented Feb 5, 2020

Emitting a bunch of metrics which are all zero strikes me as wasteful, though this is by no means the only place that Cortex does it.

@stale stale bot removed the stale label Feb 5, 2020
@jtlisi
Copy link
Contributor

jtlisi commented Feb 5, 2020

Ever since #1571, all of the rule group metrics in cortex have been populated. I think this issue can be closed. I agree there is room for improvement on null metrics generally.

@bboreham
Copy link
Contributor Author

bboreham commented Feb 6, 2020

Nit-picking, the exact metrics named are no longer exported at all by Cortex, but prefixed ones like cortex_prometheus_rule_group_duration_seconds are now exported per-user. Thanks!

@bboreham bboreham closed this as completed Feb 6, 2020
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

No branches or pull requests

2 participants