medic / helm-charts

GNU Affero General Public License v3.0
0 stars 0 forks source link

Note down helm chart version in the values.yaml for cht-4x charts #22

Open henokgetachew opened 3 months ago

henokgetachew commented 3 months ago

When the helm chart progresses and there is a change in the values.yaml structure, old values.yaml files lying around for the old charts could sometimes be used with the new charts causing confusion. Noting this version down in the values.yaml may help to debug issues.

Another scenario to investigate might be to use cht-deploy to validate the values.yaml and its compatibility.

mrjones-plip commented 3 months ago

we should be sure to update EKS docs, which cite this values file as the one to copy from.

My vote is to empty out the contents of any out of date YAML files with something like:

# please see more current file: https://github.com/medic/helm-charts/blob/main/charts/cht-chart-4x/values.yaml

this way we folks know where to go and won't try to keep using an out of date file