-
Notifications
You must be signed in to change notification settings - Fork 811
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
Add ingress.ingressClassName config option #2403
Add ingress.ingressClassName config option #2403
Conversation
ee5d168
to
41a0fec
Compare
41a0fec
to
b92d86c
Compare
We are quite low on review capacity, so I did a delayed self-review and will go for a merge now that its validated against a k8s 1.22 api-server thanks to #2404 |
jupyterhub/zero-to-jupyterhub-k8s#2403 Merge pull request #2403 from consideRatio/pr/add-ingress.ingressClassName-config
@consideRatio did you manage to test this? I'm hitting the exact same error.
|
@pebabion when using what version? This is not yet released with a non-prerelease version so you have to explicitly specify for example |
@consideRatio thanks! I figured that |
Closes #2380 that contains details about the need for this PR.