pangeo-data / pangeo-cloud-federation

Deployment automation for Pangeo JupyterHubs on AWS, Google, and Azure
https://pangeo.io/cloud.html
59 stars 32 forks source link

Telemetry for aws hub #634

Open TomAugspurger opened 4 years ago

TomAugspurger commented 4 years ago

The gcp-specific parts in https://github.com/pangeo-data/pangeo-cloud-federation/pull/629/files and https://github.com/pangeo-data/pangeo-cloud-federation/pull/626/files should probably be moved to pangeo-deploy/values.yaml to enable it for other hubs (AWS) rather than just the GCP one.

(cc @scottyhq)

scottyhq commented 4 years ago

thanks @TomAugspurger!

should probably be moved to pangeo-deploy/values.yaml to enable it for other hubs (AWS) rather than just the GCP one.

@salvis2 - can you take a pass at this?

salvis2 commented 4 years ago

Yeah I can do that.

salvis2 commented 4 years ago

@TomAugspurger you are still doing some modifications to how Prometheus / Grafana are set up on the Google hub. Should I wait until you've settled on a nice configuration for that, or just move a lot of the settings to values.yaml?

TomAugspurger commented 4 years ago

Yeah, sorry hold on this for now. I'll hopefully have something later today.

The tldr is that we'll have a separate helm deployment / namespace for metrics I think.

On Thu, Jul 2, 2020 at 12:55 PM Sebastian Alvis notifications@github.com wrote:

@TomAugspurger https://github.com/TomAugspurger you are still doing some modifications to how Prometheus / Grafana are set up on the Google hub. Should I wait until you've settled on a nice configuration for that, or just move a lot of the settings to values.yaml?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pangeo-data/pangeo-cloud-federation/issues/634#issuecomment-653144765, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKAOIWI6C6J7227IEH4S7DRZTCYZANCNFSM4OMSXRNA .