-
Notifications
You must be signed in to change notification settings - Fork 528
Improve ClickHouse Keeper Grafana Dashboard #1872
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
Conversation
… fix-keeper-dashboard
Slach
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for contribution, really appreciated to sync metrics names
in our prometheus scraping config we use pod_name, not pod
please use following commands to check your dashboard changes
minikube delete
minikube start
kubectl create ns test
helm install -n test clickhouse-operator ./deploy/helm/clickhouse-operator/
NO_WAIT=1 bash -xe ./deploy/prometheus/create-prometheus.sh
NO_WAIT=1 bash -xe ./deploy/grafana/grafana-with-grafana-operator/install-grafana-operator.sh
NO_WAIT=1 bash -xe ./deploy/grafana/grafana-with-grafana-operator/install-grafana-with-operator.sh
kubectl apply -f ./docs/chk-examples/01-chi-simple-with-keeper.yaml
|
@Slach ah i did miss the relabel config ... ok with that it works, correct. However i think this is kind of |
Slach
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
too many changes which not related to dashboard improvements
thanks to adopt script to MacOS
But please don't break name conversion, look to initial prometheus scraping config and rollback to pod_name and container_name
… fix-keeper-dashboard Signed-off-by: Slach <bloodjazman@gmail.com> # Conflicts: # deploy/prometheus/prometheus-alert-rules-chkeeper.yaml
… fix-keeper-dashboard
|
@Slach i agree; I just added a fix for the dashboard to load the prometheus datasource correct; It works now with the relabel config you pointed me to. |
… fix-keeper-dashboard

Fix ClickHouse Keeper Grafana Dashboard
fix #1844
Problem
The ClickHouse Keeper dashboard was not working correctly due to:
pod_namevspod,container_namevscontainer)Changes
Label Fixes
pod_name→podin all queries and template variablespod, notpod_namecontainer_name="clickhouse-keeper"filters from all panel queriescontainer, notcontainer_name{{pod_name}}to{{pod}}across all panelsTemplate Variable Improvements
Before:
After:
Benefits: