Skip to content
This repository has been archived by the owner on Jan 11, 2023. It is now read-only.

Update grafana and use preinstalled helm #2628

Merged
merged 1 commit into from
Apr 9, 2018

Conversation

martell
Copy link
Contributor

@martell martell commented Apr 7, 2018

READY

@martell
Copy link
Contributor Author

martell commented Apr 8, 2018

Tested this last night and everything is working

cadvisor-8cf5z                                              1/1       Running   0          19h
cadvisor-bjmxs                                              1/1       Running   0          19h
dashboard-grafana-5d864656b8-sjtbn                          1/1       Running   0          19h
monitoring-prometheus-kube-state-metrics-6bc48cd465-p76fz   1/1       Running   0          19h
monitoring-prometheus-node-exporter-gl9th                   1/1       Running   0          19h
monitoring-prometheus-node-exporter-kdlf5                   1/1       Running   0          19h
monitoring-prometheus-pushgateway-d4f679b7-trcgm            1/1       Running   0          19h
monitoring-prometheus-server-75bb797794-js8kb               2/2       Running   0          19h

Same instructions apply and it picks up the Prometheus metrics.
v5 looks like a very nice visual refresh I must say.

This is now ready for merging.

@jackfrancis
Copy link
Member

@ritazh FYI, may I lean on you for comment/approval?

@ritazh
Copy link
Member

ritazh commented Apr 9, 2018

@jackfrancis Yes I can take a look and test it.

cc @trstringer

@ritazh
Copy link
Member

ritazh commented Apr 9, 2018

LGTM
Confirm this works on 1.8.11, 1.9.6, 1.10.0

Copy link
Member

@jackfrancis jackfrancis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @ritazh!

@jackfrancis jackfrancis merged commit a0ad2d7 into Azure:master Apr 9, 2018
@martell
Copy link
Contributor Author

martell commented Apr 10, 2018

Thanks for reviewing and confirming @ritazh

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants