Inconsistent usage of metadata.namespace in chart templates

[egoldblum]

Ran into this today when using helm template to generate YAML to be subsequently applied with kubectl.

Today only connect-inject-serviceaccount.yaml and sync-catalog-service-account.yaml appear to be setting metadata.namespace via {{ .Release.Namespace }}

Ideally, helm would do this automatically when running template --namespace=<x> to mirror the behavior of helm install --namespace=<x>, but it does not. See: helm/helm#3553 for a longer discussion.

Other projects, such as Istio, have worked around this limitation by including metadata.name on all resource templates. See: istio/istio#4606

[egoldblum]

Thoughts on this change? If this is an acceptable change to workaround the helm template limitation, I'd be willing to contribute a PR.