-
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
Support Kubernetes 1.32 #2416
base: master
Are you sure you want to change the base?
Support Kubernetes 1.32 #2416
Conversation
7cbdb78
to
7cb3d06
Compare
7cb3d06
to
3309908
Compare
Signed-off-by: Jacob Salway <[email protected]>
3309908
to
672e911
Compare
@@ -1,7 +1,7 @@ | |||
run: | |||
# Timeout for analysis, e.g. 30s, 5m. | |||
# Default: 1m | |||
timeout: 1m | |||
timeout: 2m |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only takes 22s to execute on my local but was timing out in CI https://github.com/kubeflow/spark-operator/actions/runs/13088329194/job/36522115615?pr=2416
/assign @vara-bonthu @ImpSy @ChenYi015 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/approve
Does this mean users with older versions of K8s (1.28) needs to stick with old tags when this is merged?
Have all tests been executed with these updates, and are there any known issues or regressions identified?
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: vara-bonthu The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Purpose of this PR
Support Kubernetes 1.32
k8s.io/...
dependencies tov0.32.0
controller-runtime
,envtest
andcontroller-tools
to keep in sync with their upstreamk8s.io/...
compatibilitycontroller-runtime
functions to fix build errorsChange Category
Checklist