-
Notifications
You must be signed in to change notification settings - Fork 12
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
feat: updated Promethus dashboard #7
Conversation
Signed-off-by: Martin Chodur <[email protected]>
Hi, sorry for the delay. Hm.. Yeah this is unfortunate, Grafana does not provide any way of documenting the import process. To explain what is it used for. The dashboard needs to distinguish Prometheus instances from each other. It uses the instance label but if for example you have multiple prometheis instances each scraping itself they would all have value of the The dashboard loads all values of the instance label and of the label specified as So.. I see you have no options in the prtscr of custom label, which means that you have set the Default should be job since that has every metric added by Prometheus or try setting it to instance. But I have to think of a way to document it or make more robust. Thanks a lot for testing! |
Signed-off-by: Martin Chodur <[email protected]>
So I found a time to look at the issue more and found out that you probably use dashboard provisioning right? There is bit of an issue with exporting dashboards which does not show up when importing using GUI but it does when using provisioning which I managed to reproduce. The issue grafana/grafana#10786. So now hopefully it should work for you |
This is awesome @FUSAKLA !!!! |
One thing though: The rule valuation errors are not there anymore. But i guess people want to know, if the rule evaluations fail and for what reason. Would be good to bring it back. |
Hi, those are great news! :) Ad rule evaluation errors Ad 0.95 percentile |
Absolutely, yeah!
Understood. I wasn't aware it's a limitation of the metrics. I assumed it was a simple change in the collection of available quantiles in grafana. Ready for Prod, I'd say :) |
Signed-off-by: Martin Chodur <[email protected]>
Great, I'll release it ASAP just found some issues with importing still. Could you please verify hopefully for the last time this works for you? :) Just that you can import it |
@FUSAKLA: I found another little glitch. The top left graph does not use the |
Thanks, I'll fix the datasource right a way. Yes, the error panel legend is a bit unfortunate, since it shows multiple different metrics each having different labels I can't use the legend formatting. Removing the instance and custom labels from it is definitely possible but then when you will have selected multiple Prometheus instances you won't know which is having these issues and would have to go through all of them one by one to find it out. Not sure about the trade off 😕 |
Signed-off-by: Martin Chodur <[email protected]>
Signed-off-by: Martin Chodur <[email protected]>
@christoph-buente thanks a lot for testing and feedback! Much appreciated :) |
resolves #5
@christoph-buente I ended up doing bigger refactoring of the dashboard.
PTAL if it works for you, possibly I'd be glad for any opinions on it.
Mainly it now supports displaying data for multiple instances at once.
Also there is whole bunch of new information I managed to find in them metrics which I believe are useful so you will see.
The new JSON can be found here