-
Notifications
You must be signed in to change notification settings - Fork 14
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
Alerting: Failed to evaluate queries and expressions: input data must be a wide series but got type long (input refid) #168
Comments
Is anybody able to check if this is a bug or if I'm doing something wrong? |
Hey @Boeller666, |
These are the data types:
and a csv is attached |
Hey @Boeller666, We will fix this soon anyways. Thanks for reporting. |
Hmm, have to check if it only occurs if there is no data returned. Will tell you later! |
Yes, seems it will not fail if data is returned. Is there any other PR, branch or whatever to link here that solves this issue? |
@Boeller666 nope, this is not picked up yet. We had something similar in the core sql data sources recently https://github.com/grafana/grafana/pull/67844/files#diff-fb30141c4ef5bef3c02a17014449038daac2b593e7cc4e066fb3c35243a4e7dcL317 the cause of the issue is that we return the column names with null values and alerting doesn't consider this an empty dataframe. The fix we did for the sql data sources is to remove the columns if they are empty. It looks like a proper fix would be not to return the column names from upstream. From here https://github.com/grafana/grafana-plugin-sdk-go/blob/main/data/sqlutil/sql.go |
( as this relates to the no-results case, this may be similar to grafana/grafana#82306) |
I tryed to create an alert from a query that already works in explore, even as table or time series:
When using this as a table I got:
When using this as a time series I got:
Maybe related to grafana/grafana#48072 ?
The text was updated successfully, but these errors were encountered: