-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
chore(deps): Update client-go and controller-runtime #11473
base: master
Are you sure you want to change the base?
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Hi @mprahl. Thanks for your PR. I'm waiting for a kubeflow member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
5887e81
to
168310f
Compare
Sorry about the noise on this one. I had some issues with the license check but upgrading go-license fixed the segfault that was being encountered. /cc @HumairAK |
It seems that the "KFP Samples / KFP Samples - K8s v1.31.0" workflow flaked. Can someone please retry it? |
I reopened the PR to restart the workflows. |
a933ad1
to
88df99e
Compare
This is now rebased and it also includes the update to the crypto library like in #11482 |
util.NewFakeTimeForEpoch(), | ||
"PERSISTENCE_WORKER", | ||
eventHandler, | ||
false, | ||
saver) | ||
assert.NoError(t, err) |
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.
Can we change it to require.NoError(t, err)
? It seems like there's no point in continuing the execution in case of error.
Can you do that for the other assertions too?
This updates k8s.io/client-go to v0.30.1 and sigs.k8s.io/controller-runtime to v0.18.6. Updating any higher requires updating github.com/argoproj/argo-workflows/v3 which requires updating to Go 1.23 which isn't widely available in base container images yet. Some other dependencies were updated for dependency resolution to succeed. Signed-off-by: mprahl <[email protected]>
88df99e
to
12331b0
Compare
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.
/lgtm
/ok-to-test
Description of your changes:
This updates k8s.io/client-go to v0.30.1 and sigs.k8s.io/controller-runtime to v0.18.6. Updating any higher requires updating github.com/argoproj/argo-workflows/v3 which requires updating to Go 1.23 which isn't widely available in base container images yet.
Checklist: