-
Notifications
You must be signed in to change notification settings - Fork 8.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Chart: Creating extraManifests on Helm Chart #11370
Conversation
This issue is currently awaiting triage. If Ingress contributors determines this is a relevant issue, they will accept it by applying the The Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Hi @msfidelis. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
✅ Deploy Preview for kubernetes-ingress-nginx canceled.
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: msfidelis The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
fix(chart): lint fix(chart): typo fix(chart): typo fix(chart): template rendering fix(chart): template rendering fix(chart): helm chart test fix(chart): helm chart test fix(chart): helm chart test name
3164f4d
to
57fd73e
Compare
I cannot recall where, but we already had requests like that in the past and decided not to support them as they bring extra complexity to the chart which should be covered by user's tooling. Anything could be put into that value and users will come up with issues and asking for our support just because we are offering this possibility. It's basically adding a wildcard to add any kind of manifest, even those we do not know yet, and users will ask us to extend that feature to a point where we cannot support it anymore. It's already complicated enough to support manifests not directly related to how Ingress NGINX works right now. I'm closing both this PR and the issue now. |
/close |
@Gacko: Closed this PR. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Ah, yeah, here we go. This is the other PR: #10945 |
I understand you point @Gacko , thanks! |
What this PR does / why we need it:
I'm sending this Pull Request to address the Issue that mentions the helm chart's ability to create extra manifests on demand. I think the issue may require a bit more screening, but I'm putting the work in progress to discuss.
Types of changes
Which issue/s this PR fixes
fixes #11351
How Has This Been Tested?
Checklist: