Vonage / Grafana_Status_panel

A panel plugin for Grafana to monitor multiple parameters at once
Other
178 stars 108 forks source link

How can I add data source support? #66

Open Louis-7 opened 7 years ago

Louis-7 commented 7 years ago

Hi Guys,

I want to know how can I support other data source? Currently we are using OpenTSDB as data source, I've tried this plugin in my project, seems it works with OpenTSDB. Could someone tell me for what reason it's not supported ? If I know the reason I may help on this.

tomer-amir-vonage commented 7 years ago

Hi!

I wrote the explanation on a few different issues. you can find them in #10 and #17

We'll be happy if you could help with it and make a pull request. Thanks!

Louis-7 commented 7 years ago

Thanks! @TomerAmirV

I'm using OpenTSDB as data source, I've try several times to send multiple queries and the result return just like the order I set in "Metrics" tab. Maybe Grafana team already solve it? You can check from your side.

But, I still don't got why this bug block add other data source in this plugin, if as you said the order is not right, why influxDB & Graphite don't have the issue?

Louis-7 commented 7 years ago

As I understand If we want to support other data source, what should we do is use your plugin with that data source and do some test to see if there is no bug? Am I right? If I'm wrong please correct me, thanks!

tomer-amir-vonage commented 7 years ago

@Louis-7, I am not sure I understand what you mean, but basically, if all the features seem to work, including the templated queries, then it should be fine.

Louis-7 commented 7 years ago

@TomerAmirV . Thanks for your answer. If I'm not misunderstanding the bug you mentioned, this plugin is support OpenTSDB. I guess grafana use ‘TimeSeries’ class convert all time series data source to grafana standard format, so different data source won't be a problem.

Actually, I didn't got the 'templated queries problem' 100% even after I read the answer from the link you given me.