[FIX][NAMESPACE]:Add namespace injection to all Helm templates using .Release.Namespace#394
Conversation
|
@devlopergalxboy thanks for the PR! Could you please let us know what exact problem it solves? AFAICT it's already possible to install Zammad into namespaces via |
@mgruner when we tried to deploy using rendered manifest pattern we identified that most of our resources were not being injected with the correct namespace which left us out of sync on our ArgoCD as such we thought it would be a great idea that who ever wants to use zammad with the rendered manifest pattern they can do so without having issues of namespace injection. Yes adding the flag --namespace to the command works well. |
|
@monotek do you agree to this change? |
monotek
left a comment
There was a problem hiding this comment.
That's exactly the reason I don't like ArgoCD.
I just won't work without changes to the resources it should just manage 🙄
FluxCD FTW! ;-)
But in the end the change does no harm.
|
Thank you @devlopergalxboy! |
What this PR does / why we need it
Summary of this PR
Implements namespace injection in all Helm templates using .Release.Namespace to ensure resources are deployed in the intended namespace.
Related Issues
(If applicable, reference issues in the format: closes #123)
Additional Information for Reviewers
Please let me know if further changes or clarifications are needed.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]