-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Allow the Chart to create extra manifest #11351
Comments
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. |
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 issue and the PR now. /close |
@Gacko: Closing this issue. 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 |
Sometimes, accessories to the installation may be manifests to be installed (for example, NetworkPolicies or Ingresses).
It would be nice to be able to do this directly via the Chart using the approach that Grafana, Prometheus or OAuth2 Proxy also uses.
Ref: helm/helm#12653
The text was updated successfully, but these errors were encountered: