-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
Stop vendoring heapster v1alpha1 #45758
Stop vendoring heapster v1alpha1 #45758
Conversation
/release-note-none |
6d9d443
to
9fbf4e7
Compare
9fbf4e7
to
1130989
Compare
@piosz @DirectXMan12 ptal. Thanks! |
1130989
to
1d97347
Compare
LGTM assuming tests pass |
Thanks. All tests passed. Adding lgtm per @DirectXMan12 's comment. |
/lgtm |
@caesarxuchao: you cannot LGTM your own PR. In response to this:
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. |
@wojtek-t could you help approve the change? Thanks. |
@caesarxuchao - please squash commits a bit (I would suggest merging 1st and 4th and separately 2nd and 3rd - those two seem to be auto-generated). /approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: caesarxuchao, wojtek-t
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
1d97347
to
27e2129
Compare
@wojtek-t squashed as suggested. Thanks. |
vendor/k8s.io/heapster/metrics/apis/metrics/v1alpha1 fix some related imports
hack/update-godep-license
27e2129
to
36e2d0b
Compare
Rebased. |
@caesarxuchao: The following test(s) failed:
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. |
Automatic merge from submit-queue (batch tested with PRs 44337, 45775, 45832, 45574, 45758) |
Automatic merge from submit-queue remove exception in verify-no-vendor-cycles.sh The exception is fixed by #45758.
Changes to use the one in staging/metrics.
TODO: remove the exception in #45176
Implementing #45498 (comment).