-
Notifications
You must be signed in to change notification settings - Fork 18
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
alerts fail to evaluate queries and expressions #173
Comments
Hi @mhristof thanks for opening this issue! I'm having some trouble recreating this but I think my test dataset is just different and it's hard to match the query to catch this edge case. Would you mind showing us a screenshot of what the query results look like in table format when on the query editor/explore page with any sensitive data removed so we might try to recreate this on our end? Thanks so much! |
@mhristof that's very helpful thanks! Still haven't quite recreated it yet, but I wonder if the issue is related to the missing values. I see we have this additional option for missing values where they can be set to a specific value such as zero: Does altering that number help? |
It's not the same error message as what you're seeing but I am seeing inconsistent data in general between alerts and queries. Made another issue to track that here: #176 |
Setting
Do you have any idea why we are getting the emty/null values in the results. If i use the same query in Kibana, only the non empty values are returned (see https://stackoverflow.com/a/70439328/2599522 for example) |
I'm so sorry that fix didn't work for you! I think it's likely we have some kind of bug in how our plugin is handling alerting queries, particularly around nil values, but even more generally I'm seeing inconsistent results between the results in our query editor and the results in our alerts. This is likely because unlike many of our other datasource plugins, this one seems to have 2 different query paths, one for alerting and one for the query editor. In our other plugins, we tend to have the same data flow path for both, which ensures any bug fixes in one area are fixed in the other. We're currently scoping out a refactor of this plugin to unify the experience between alerting queries and query editor queries, and it's one of our top priorities for the quarter. But unfortunately it may take us at least a few weeks before we can ship improvements, though we will do what we can to expedite this process! I am curious has alerting with this datasource worked well for you in the past and recently started showing errors? Just want to rule out that this isn't related to some sort of recent changes we may have made on our end. I suspect not and that these issues have likely been around for a long time, but would like to rule that out in case there's a quick fix, we can get out to unblock you. |
I've only recently started to add alerts as this is a fresh grafana instance/opensearch integration. Thanks for the updates, you've been most helpful. At the moment we can live without the alerts. Let met know if you need testing when beta versions are out |
Great thanks @mhristof will do! cc @fridgepoet and @idastambuk for your refactor project! |
I was previously able to create alerts with this plugin using the following versions. Grafana: 9.4.2 I recently upgraded to Opensearch Plugin 2.4.1 and Grafana 9.5.2. I upgraded the plugin first, but don't recall testing to see if I could still create alerts. I can say that as of today with both recent upgrades I've done; I cannot create alerts and get the same error message as seen in the original post. |
Hey @maxwellvarner can you try plugin v2.6.1? There might be a backend client creation issue in v2.4.1 that you're running into. |
@fridgepoet thank you for that suggestion. I have just upgraded to v2.6.1 and am able to create alerts again! |
@mhristof Can you also try plugin v2.6.1? There might be a client issue in v2.4.1 that you're running into. |
Hi @pavriet-boxtal, thanks for reporting this. I just have a few questions:
Thank you! |
Hi @idastambuk, In the picture bellow, I was able to collect the same query (in a panel editor this time) where an error happened (blue vertical line @16:07) + the query inspect menu on the right. Just to show that data does not seem to be the issue.
|
Hi @pavriet-boxtal, thanks for all the information so far. What version of the plugin were you using when you did not have any issues? |
Hello @fridgepoet , We had issues with both 2.4.1 and 2.6.2. We never had no issues :( |
Hi @pavriet-boxtal, can you by any chance recall if the error for 2.4.1 was the same as in 2.6.2 ( |
I search our notification channels for EOF and found this (url anonymized for obvious reasons). I check and it happened twice while on 2.4.1. I also attached a picture of the alert query that generated the errors.
These errors are very rare compared to the |
v2.6.1 made my issue go away. I can now see the data in my queries and they are not failing. |
@mhristof thanks for the update! I will close this ticket as it looks like the original issue was fixed with 2.6.1. @pavriet-boxtal I opened another issue to track the problem from your comment, you can follow the progress there and let us know if you find any other errors, as you mentioned. |
What happened:
when trying preview an alert, im getting the following error
What you expected to happen:
The alert to preview the values
How to reproduce it (as minimally and precisely as possible):
Create an alert as above and try to preview the status
Anything else we need to know?:
Environment:
The text was updated successfully, but these errors were encountered: