Helm chart should template namespace

[ashlineldridge]

Is your feature request related to a problem? Please describe. Yes, same as this issue for the agent operator, it would be nice if the chart templated the namespace associated with namespaced resources - e.g. like the tempo chart does.

metadata:
  namespace: {{ .Release.Namespace }}

Currently it doesn't which means that running helm template --namespace does not have the effect of producing correctly namespaced resources. This makes setting up GitOps repositories more cumbersome.

Describe the solution you'd like Namespaced resources and references to them should use the {{ .Release.Namespace }} syntax.

Describe alternatives you've considered Running kustomize over the templated output to patch the namespace in.

Additional context No.

[lucasoares]

Would be nice to use {{ default .Values.namespace .Release.Namespace }} instead! Want to be able to deploy it to a different namespace...

[jeschkies]

@ashlineldridge or @lucasoares would you propose the change in a PR? I think it's reasonable but we won't have the time to work on it.

[lucasoares]

@ashlineldridge after this PR: https://github.com/grafana/loki/pull/9330 loki already uses Namespace throughout the entire helm chart.. I think my needs is different, since I need to have a namespaceOverride value to be able to override the default namespace... I will create a PR with this.

[lucasoares]

@ashlineldridge or @lucasoares would you propose the change in a PR? I think it's reasonable but we won't have the time to work on it.

May you please feedback me on the PR I just created?