-
Notifications
You must be signed in to change notification settings - Fork 2k
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
[release-2.0] Update go to 1.16, update dependencies, remove vendor directory #1419
Conversation
k8s.io/apimachinery v0.18.3 => k8s.io/apimachinery v0.18.3 | ||
k8s.io/client-go v0.18.3 => k8s.io/client-go v0.20.0 | ||
k8s.io/api v0.18.3 => k8s.io/api v0.20.4 | ||
k8s.io/apimachinery v0.18.3 => k8s.io/apimachinery v0.20.4 |
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.
@lilic not sure if you wanted to keep apimachinery on 0.18.3 on purpose or this change is okay
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.
Most likely a mistake on my side, 0.20.4 sounds great!
Signed-off-by: Kemal Akkoyun <[email protected]> Signed-off-by: Manuel Rüger <[email protected]>
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
Thank you! Do you mind merging these changes into master as well?
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: lilic, mrueg 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 |
* [CHANGE] Rename --labels-metric-allow-list to --metric-labels-allowlist kubernetes#1424 * [CHANGE] Remove deprecated Kubernetes APIs kubernetes#1423 * [CHANGE] Remove vendor folder kubernetes#1419
* [CHANGE] Rename --labels-metric-allow-list to --metric-labels-allowlist kubernetes#1424 * [CHANGE] Remove deprecated Kubernetes APIs kubernetes#1423 * [CHANGE] go.mod: Update Dependencies kubernetes#1419 * [CHANGE] Remove vendor folder kubernetes#1419 * [CHANGE] `k8s.gcr.io/kube-state-metrics/kube-state-metrics` becomes the authoritative registry Location on quay.io will not be updated anymore. Previously pushed images will be kept around to avoid breaking existing deployments.
What this PR does / why we need it:
This changes the used go version in the build process to 1.16 and updates various dependencies.
It also removes the /vendor directory