Vonage / Grafana_Status_panel

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

BUG: Panel Title Link does not take into account the time range and the variables #96

Open pparth opened 6 years ago

pparth commented 6 years ago

Hello,

As far as i understand, the Status Panel Title URL is equal to the first Detail Link added to the panel menu. However, it does not seem to take into account the selected time range and more importantly the variables. If this is not possible, i prefer the ability to completely disable the Title URL, as it causes confusion and buggy behaviour.

pparth commented 6 years ago

Anything new about this please?

YehonatanVonage commented 6 years ago

Hi @pparth , Thanks for your feedback, it helps us to improve our grafana plugin. You can add to the url params (in "Url params" field) some get params with the format: varName1=$varValue1&varName2=$varValue2 (the variables should be configured in the "Templating" panel). Currently adding the time to the url is not supported. Unfortunately, I won't be able to add this feature in the near future, and I remarked it as enhancement. If it is urgent, you can make a pull request for this, and I'll approve it.

pparth commented 6 years ago

Maybe i am missing something here. I do not see any "Url Params" under the Options --> Panel Options section. Are you talking about General --> Drill Down/Detail Link? These work ok, my menu items work ok. I do use Variables and i have Variables ticked. But the variables are NOT used when constructing the Panel Title link. I can use the urlParams, but then i need to untick variable usage because then i get the url parameters twice and this breaks things.

pparth commented 6 years ago

Ok, i unticked the Variables and adde them as url params. Everything works as expected. This is a quick workaround indeed. However, the time params are not working as you mentioned and it is also best for the variables to work as expected. Other than that, good work!!!