-
Notifications
You must be signed in to change notification settings - Fork 4.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 dependencies to kubernetes 1.16 #8055
Conversation
c8d9afb
to
0aa29d1
Compare
86a8832
to
d8802c1
Compare
d8802c1
to
caa8180
Compare
caa8180
to
f1fef78
Compare
/test pull-kops-e2e-kubernetes-aws Looks like a flake |
@@ -32,7 +32,8 @@ import ( | |||
|
|||
"k8s.io/client-go/kubernetes" | |||
"k8s.io/client-go/rest" | |||
_ "k8s.io/kubernetes/pkg/client/metrics/prometheus" // for client metric registration | |||
|
|||
// _ "k8s.io/kubernetes/pkg/client/metrics/prometheus" // for client metric registration |
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.
I tracked down the PR that moved this package
https://github.com/kubernetes/kubernetes/pull/81432/files#diff-a9f93108ac59ff44885fc0aea2f29e71R29
and it looks like this can be updated to k8s.io/component-base/metrics/prometheus/restclient
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.
Thanks - great find! Replaced with that new location
4d3e5ed
to
4c5efd9
Compare
/test pull-kops-e2e-kubernetes-aws |
2 similar comments
/test pull-kops-e2e-kubernetes-aws |
/test pull-kops-e2e-kubernetes-aws |
for whenever you're ready to remove WIP |
Helps pare down our dependency on kubernetes/kubernetes
Another dependency on k/k removed! prometheus
klog added a flag (add_dir_header).
Required for static-check to pass.
4c5efd9
to
5ebbfc9
Compare
/retest Strange that we're getting image not found errors, I'd expect the version markers that we use to indicate that the images have been pushed already.
|
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: justinsb, rifelpet 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 |
@justinsb: The following test failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. 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. I understand the commands that are listed here. |
Helps pare down our dependency on kubernetes/kubernetes