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
Unify towards k8s.io/utils/clock - part 2 #105026
Conversation
/triage accepted |
cd5512b
to
47446f6
Compare
47446f6
to
3b8fb54
Compare
597505f
to
3c34aba
Compare
3c34aba
to
5f72d78
Compare
/retest |
@@ -35,6 +35,7 @@ | |||
allowedImports: | |||
- k8s.io/apimachinery | |||
- k8s.io/kube-openapi | |||
- k8s.io/utils/clock |
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.
Please double check this one :)
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.
Seems fine!
/assign @lavalamp Please take a look when you find a moment - second part of mechanical changes (modulo one commit where I update import restrictions - please double check that). The third PR I'm planning to do after this one should finalize this migration. |
5f72d78
to
b7221bc
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: lavalamp, wojtek-t 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 |
The Kubernetes project has merge-blocking tests that are currently too flaky to consistently pass. This bot retests PRs for certain kubernetes repos according to the following rules:
You can:
/retest |
Ref #94738
/kind cleanup
/priority important-longterm