pangeo-data / helm-chart

Pangeo helm charts
https://pangeo-data.github.io/helm-chart/
21 stars 26 forks source link

Add Grafana and Prometheus to Chart #88

Closed jhamman closed 3 years ago

jhamman commented 5 years ago

closes #6

Just getting this started. Hoping to crowd source the specific configuration values in monitoring.yaml.

jacobtomlinson commented 5 years ago

This is an interesting one. I agree with your approach to stubbing out the config for the monitoring. However by including the chat dependencies it will use their default values and install them.

We may need to research more into whether you can make dependencies configurable and optional.

guillaumeeb commented 5 years ago

Looks like this is feasible: https://github.com/helm/helm/blob/master/docs/charts.md#tags-and-condition-fields-in-requirementsyaml

jhamman commented 5 years ago

@guillaumeeb - do you want to take a hack at the optional dependencies configuration?

guillaumeeb commented 5 years ago

I can and would like to:

I've no access to any GCP account currently...

guillaumeeb commented 5 years ago

@jhamman I'm hoping to have some time to work on this in april. So should I work directly from https://github.com/pangeo-data/pangeo-cloud-federation or manually deploy things in some test GCP cluster?

jhamman commented 5 years ago

@guillaumeeb - we can certainly get you setup on one of our GCP clusters to test things out. We could also just use a minikube deployment. Let me know what your preference is and we'll get you going.

guillaumeeb commented 5 years ago

Didn't thought about minikube! Have some of you already tested Pangeo with it? Will it be representative enough?

jhamman commented 5 years ago

@guillaumeeb - Yes, I think minikube should work just fine. If you send me your Google credentials, I'll also get you setup on GCP.