canonical / alertmanager-k8s-operator

https://charmhub.io/alertmanager-k8s
Apache License 2.0
5 stars 17 forks source link

fix(dashboards): use rate_interval instead of interval #278

Closed lucabello closed 1 month ago

lucabello commented 1 month ago

Issue

Closes #274.

Solution

Use $__rate_interval instead of $__interval.

mmkay commented 1 month ago

One thing to potentially consider here: https://hoelz.ro/blog/use-caution-when-using-rate_interval-along-with-increase

This might change the values scale visible in the dashboards. Might be surprising to some?

I wonder if we shouldn't change the variable only in the panels that use rate(.

lucabello commented 1 month ago

You're right, that replace was too lazy :) Now we only changed to $__rate_interval in rate() queries.