helm chart changing meaning of ingress.enabled was unexpected #2372
Closed
roderickgreen
started this conversation in
General
Replies: 1 comment 2 replies
-
|
Thanks for your feedback! Let me recheck on this |
Beta Was this translation helpful? Give feedback.
2 replies
-
|
@roderickgreen it is back in chart version 0.35.0 |
Beta Was this translation helpful? Give feedback.
-
|
Thank you @VietND96! |
Beta Was this translation helpful? Give feedback.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Just general feedback: changing the meaning of ingress.enabled from "create an ingress resource" to "install an ingress controller" cost me a decent amount of time to track down. I'd personally reconsider that change because most charts in the wild use ingress.enabled to create an Ingress resource.
Beta Was this translation helpful? Give feedback.
All reactions