GoogleCloudPlatform / prometheus-engine

Google Cloud Managed Service for Prometheus libraries and manifests.
https://g.co/cloud/managedprometheus
Apache License 2.0
191 stars 89 forks source link

refactor(helm): remove updateversion and centralized one value file. #953

Closed bwplotka closed 4 months ago

bwplotka commented 4 months ago

Changes

Rationales

Open Questions

bernot-dev commented 4 months ago

Completely agree that we now have better options than sed to update versions more safely.

Overall, I like this change. It's not clear to me that putting all of the images in a global file is the best solution, because it has to be manually included, and not all of the charts use all of those images. But it's probably fine because we're reducing the number of calls to helm template. And centralizing versions into one file has advantages, as well.

Since the versions are not included in the default values file now, I would consider making them required to make it noisier if the version is not supplied.

bwplotka commented 4 months ago

Addressed comments and simplified to one values.global.yaml -- simpler and consistent.