fix(helm): explicitly include namespace in manifests

[valorl]

Currently, most of the manifests omit .metadata.namespace.

This works fine with helm install / helm upgrade where all the manifests without a namespace will be installed to --namespace anyway.

However, helm template will produce the manifests as-is (with no namespace), regardless of --namespace, forcing users to add the namespace manually or use more tooling.

This change adds the namespace to all manifests.

Fixes #1607

PR Description

Which issue(s) this PR fixes

Notes to the Reviewer

PR Checklist

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

[CLAassistant]

All committers have signed the CLA.

[github-actions[bot]]

This PR has not had any activity in the past 30 days, so the needs-attention label has been added to it. If you do not have enough time to follow up on this PR or you think it's no longer relevant, consider closing it. The needs-attention label signals to maintainers that something has fallen through the cracks. No action is needed by you; your PR will be kept open and you do not have to respond to this comment. The label will be removed the next time this job runs if there is new activity. Thank you for your contributions!

[neopointer]

@clayton-cornell I'm just tagging a random contributor here. What is this PR missing to get merged?

[clayton-cornell]

@neopointer It looks like this one got missed in the shuffle. Lets see if we can get a developer/maintainer review happening here.

@grafana/grafana-alloy-maintainers can someone from the team take a look at this?

[mattdurham]

This looks reasonable to me and the namespace defaults to release.namespace so should be backwards compatible for how it was meant to be used.

[mattdurham]

@petewall any thoughts on if this causes any issue?

[mattdurham]

This PR #2044 seems to be the most active and achieves the same result?

[valorl]

Closing in favor of #2044, it's more complete and achieves the same.