-
Notifications
You must be signed in to change notification settings - Fork 380
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
Update dependency go modules for k8s v1.26.0 #797
Update dependency go modules for k8s v1.26.0 #797
Conversation
5329af9
to
3a2a001
Compare
87ced66
to
eeda108
Compare
/assign @xing-yang |
/release-note-none |
eeda108
to
e0abaca
Compare
e0abaca
to
bb5a901
Compare
fb3f750
to
e58d9d2
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: sunnylovestiramisu, xing-yang 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 |
@@ -31,11 +31,11 @@ require ( | |||
github.com/pkg/errors v0.9.1 // indirect | |||
github.com/spf13/pflag v1.0.5 // indirect | |||
golang.org/x/mod v0.6.0 // indirect | |||
golang.org/x/net v0.1.1-0.20221027164007-c63010009c80 // indirect | |||
golang.org/x/net v0.3.1-0.20221206200815-1e63c2f08a10 // indirect |
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.
@sunnylovestiramisu Can you submit another PR to make a similar change in client/go.mod as follows:
replace golang.org/x/net => golang.org/x/net v0.4.0
I'm getting a dependencies change error when generating the changelog. I wonder if it is because the client and server go.mod are having different lib version.
Ran kubernetes-csi/csi-release-tools go-get-kubernetes.sh -p 1.26.0
Does this PR introduce a user-facing change?: