-
Notifications
You must be signed in to change notification settings - Fork 43
Add support for influxdb datasource #33
Comments
@StefanSa, we are not using InfluxDB on a daily basis, so probably it's not in our nearest roadmap. Of course, we are open for contribution, and we'll be happy to help with PR. |
I may be interested in tackling this, time permitting. Any pointers from the developers on the approach to follow? Never developed for Grafana.. Sent with GitHawk |
@voiprodrigo There is an inconsistence between datasource results. Panel plugin has handler for 'data-received' event: https://github.com/flant/grafana-statusmap/blob/master/src/status_heatmap_ctrl.js#L132
So, each datapoint has timestamp in item with index |
@diafour thanks, this info should be very helpful. But I just tested on Grafana 5.4.0 and.. it actually seems to be working! |
So what, we just have to add info to the README about InfluxDB support and close this ticket? @voiprodrigo, can you confirm, that plugin is working with InfluxDB? |
Well, I did not test extensively. But I added a query on top of a suitable measurement, used the value to text mapping feature also, and that gave the expected result. From that experience, it’s working. Sent with GitHawk |
I should finally test it and fix a README. |
I am confused whether this can work with InfluxDB or not. https://github.com/flant/grafana-statusmap#prometheus says we should set up discrete values , but this InfluxDB ticket sounds as if such a query is not supported by InfluxDB ? I can only add WHERE clauses for tags in my InfluxDB database. This is also the reason why I didn't map my status strings to numbers. Maybe I should make my Status column a tag in InfluxDB? As you can see in my InfluxDB response every inverters last status is Warten , but these are not shown in the visualization? InfluxDB request/response as JSON (click to expand)
|
As i can understand, 7581 is about setups when status pushed into influxdb only when it is changed and it seems as not your case. TODO:
|
@konstantinblaesi your case is fixed by #55 , see screenshots in #53 (comment) |
Thanks for fixing this, looks great! |
InfluxDB related information is added in README. Close it. |
Thanks for the interesting plugin.
I would also like to use it, but as datasource i have influxdb.
Is there a chance that influxdb will be supported?
The text was updated successfully, but these errors were encountered: