-
Notifications
You must be signed in to change notification settings - Fork 2.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
Update kube dependencies to 1.18.6 #17869
Conversation
Blocked by knative/pkg#1387 , knatives api imports a client, that's some very impressive shit. |
go.mod
Outdated
k8s.io/apimachinery => k8s.io/apimachinery v0.17.3 | ||
k8s.io/client-go => k8s.io/client-go v0.17.3 | ||
k8s.io/code-generator => k8s.io/code-generator v0.17.3 | ||
k8s.io/api => k8s.io/api v0.18.3 |
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.
Looks like we can get rid of these replace stanzas?
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.
No, why would we?
2920b74
to
9290bba
Compare
8d90ce4
to
a750f27
Compare
/assign @amwat |
it's been removed, in #18567 |
6326b98
to
6ae19b4
Compare
01abcad
to
fbaa80d
Compare
fbaa80d
to
dfceaa5
Compare
/retest |
github.com/Azure/go-autorest => github.com/Azure/go-autorest v12.2.0+incompatible | ||
github.com/docker/docker => github.com/docker/docker v1.4.2-0.20200203170920-46ec8731fbce |
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.
Not sure why, but without the replacements, ./hack/update-deps.sh
would update the pubsub and docker dependencies, causing compilation failures.
I've pinned all the k8s dependencies to make sure we use the same version for everything
/assign fejta |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: alvaroaleman, fejta 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 |
This includes some changes to the generated clients, all their methods now take a
context.Context
as first and ametav1.<<METHOD>>Options
as last argument. Those were fixed via https://github.com/kubernetes-sigs/clientgofix or manually when we re-defined a sub interface.