fix(helm): ensure helm resources have correct namespace

[shinebayar-g]

PR Description

Helm chart resources don't have correct namespaces.

Which issue(s) this PR fixes

Issue can be reproduced by running helm template --namespace alloy alloy grafana/alloy --version 0.9.2.
Generated yaml files didn't have correct namespace.

Notes to the Reviewer

PR Checklist

• CHANGELOG.md updated
• Documentation added
• Tests updated
• Config converters updated

[CLAassistant]

All committers have signed the CLA.

[wildum]

@petewall another small helm chart PR if you have time

[petewall]

I get nervous about the removal of the use of the alloy.namespace template function. That might break some users.
Perhaps instead of using .Release.namespace, we use alloy.namespace which is enabled by the namespaceOverride value

[shinebayar-g]

I get nervous about the removal of the use of the alloy.namespace template function. That might break some users. Perhaps instead of using .Release.namespace, we use alloy.namespace which is enabled by the namespaceOverride value

Didn't seem that was the case considering that it was undocumented value and some resources were already using the {{ .Release.Namespace }} like in the operations/helm/charts/alloy/templates/pdb.yaml file.

But updated the PR regardless.

[shinebayar-g]

I resolved the conflict, please review. For now I'm using my local fork as a workaround.

[clayton-cornell]

@shinebayar-g There are some generated files that need a poke... if you run make generate-helm-docs and make generate-helm-tests, and then push the changes, the Test Helm chart tests should pass.

[shinebayar-g]

Thanks @clayton-cornell , I've updated the PR as requested and rebased.

[clayton-cornell]

@petewall @wildum Is this PR ready to approve/merge?