diggerhq / helm-charts

Apache License 2.0
8 stars 6 forks source link

Remove namespace from values #3

Closed dannysauer closed 6 months ago

dannysauer commented 7 months ago

Helm has a --namespace argument for a reason. ;)

Using a non-standard mechanism like putting the namespace in the values file "breaks" tools like argo and flux, and confuses people who expect the helm .Release.Namespace value to be respected. This change also means future expansion is easier, since there is no need to remember to add an explicit namespace to every single resource. :)