-
Notifications
You must be signed in to change notification settings - Fork 6
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
SLO Burn rate monitoring is incorrect #41
Comments
I'm wondering what happens when I think it should be guarded against. |
Looking at my specific monitoring query it seems that the
This would actually fix the query and I can see that the the burn rate would exceed the value of 1. |
thnx @lswith, its very encouraging for us to know it being useful.
great catch on both, will fix and create a new release after testing them out by end of the day. |
fix #41 : bug in monitor query template and param
Just confirmed. This fixed the issues with alerting! Thanks again |
Hi team, I've been using your tool extensively and I am loving it!
I have come across an issue with the monitoring of an SLO.
My current alerting configuration is as follows:
When I evaluate the SLO over a 24h period, it is currently at 98.41897 (which is below the 99 to meet the SLO).
I would have expected that I would receive at least 1 email stating that this SLO is not being met, however all the monitors generated aren't being triggered.
I'm wondering if the calculation of one of these items may be incorrect?
Current version: There is no slogen command to output the version, but I'm pointing to the latest of the main branch.
The text was updated successfully, but these errors were encountered: