-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[QUESTION] Supporting nginxinc ingress #2276
Comments
I assume you're referring to the ingress class name? If so the controller supports passing this as a CLI argument however I just noticed it's not exposed in the Helm chart. I'm sure there's more work we can do to support different ingress types but for the immediate case, would being able to specify the ingress class name solve your issue? I'll raise a PR to do so regardless. https://github.com/kubeflow/spark-operator/blob/master/cmd/operator/controller/start.go#L155 |
There are two different nginx ingress controllers with the same name. One is maintained by Kubernetes (what spark-operator supports) and nginx ingress maintained by the nginx company (nginxinc). Both controllers have the same ingress class name ('nginx'). What spark-operator is writing out doesn't work with |
@jacobsalway Just following up on this. I don't think what you described will fix the issue we are facing. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
spark-operator currently outputs
kubernetes/ingress-nginx
ingress. Unfortunately we usenginxinc/kubernetes-ingress
within our cluster. Since both ingress has the same name, I don't believe it can co-exist within the same cluster. Any thoughts on also supporting nginxinc's version?The text was updated successfully, but these errors were encountered: