-
Notifications
You must be signed in to change notification settings - Fork 120
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
Grafana 8.0.4 alerts does not correctly work (multiple alerts per graph) #362
Comments
thanks a lot for reporting |
Eugene, I will try :) |
Any progress so far? In Grafana 8.2 it seems you can't create alerts the old way anymore if you use the new ones, so we can't setup any alerts based on this plugin anymore. |
I found the root reason, but unfortunatelly it's not easy to fix. New grafana unified alerts need to implements full sql parser on golang part @derN3rd you can set up
|
My problem is, that we are dependent on the new alertings, so I sadly can't just disable the new alerts and use the old ones... EDIT: Just seen the link for the progress. Sorry |
Hello!
I have grafana 8.0.4 with new alerts and plugin version 2.3.1.
And there are two problems:
Alert does not contain "labels" and I can't create multiple alerts from one graph like this [Feature request] Multiple alerts per graph grafana/grafana#7832 (comment)
sometimes doubling metrics names in alerts
sample request
SELECT Time, groupArray((instance,value)) as arr FROM (SELECT toDateTime(now() - INTERVAL 60 SECOND) as Time,'cpu1_host1'as instance,20 as value UNION ALL SELECT toDateTime(now() - INTERVAL 120 SECOND) as Time ,'cpu1_host1'as instance,30 as value UNION ALL SELECT toDateTime(now() - INTERVAL 60 SECOND) as Time ,'cpu1_host2'as instance,50 as value UNION ALL SELECT toDateTime(now() - INTERVAL 120 SECOND) as Time ,'cpu1_host2'as instance,40 as value ) GROUP BY Time,instance ORDER BY Time
Is there any way to fix this?
Thank you.
The text was updated successfully, but these errors were encountered: