Skip to content
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

Manually fix the CRD validation issue in k8s 1.18 #986

Merged
merged 1 commit into from
Jul 24, 2020

Conversation

liyinan926
Copy link
Collaborator

Fixes #921.

@liyinan926 liyinan926 merged commit 5b2fe92 into kubeflow:master Jul 24, 2020
jkleckner pushed a commit to jkleckner/spark-on-k8s-operator that referenced this pull request Nov 24, 2020
PR kubeflow#1045 [1] appears to have accidentally reverted kubeflow#986 [2] which added
defaults to the protocol field to work with versions of k8s 1.18+

This PR also draws on kudobuilder/operators#297 [3]


[1] kubeflow#1045

[2] kubeflow#986

[3] kudobuilder/operators#297
@jkleckner
Copy link
Contributor

Created a candidate PR here #1084

liyinan926 pushed a commit that referenced this pull request Nov 25, 2020
PR #1045 [1] appears to have accidentally reverted #986 [2] which added
defaults to the protocol field to work with versions of k8s 1.18+

This PR also draws on kudobuilder/operators#297 [3]


[1] #1045

[2] #986

[3] kudobuilder/operators#297
jbhalodia-slack pushed a commit to jbhalodia-slack/spark-operator that referenced this pull request Oct 4, 2024
jbhalodia-slack pushed a commit to jbhalodia-slack/spark-operator that referenced this pull request Oct 4, 2024
PR kubeflow#1045 [1] appears to have accidentally reverted kubeflow#986 [2] which added
defaults to the protocol field to work with versions of k8s 1.18+

This PR also draws on kudobuilder/operators#297 [3]


[1] kubeflow#1045

[2] kubeflow#986

[3] kudobuilder/operators#297
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CRDs don't pass validation on k8s 1.18
2 participants