-
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
Introduce event clocks based on k8s.io/utils/clock #103830
Conversation
fbfec52
to
2d40e34
Compare
82451ee
to
350bced
Compare
What is wrong with this thing? I keep getting complaints about package Here, for example, is the complaint from the pull-kubernetes-verify job:
|
/triage accepted |
/cc @apelisse (you seem to have commented on similar PRs) Thanks! |
fe5d6cd
to
2341f3d
Compare
The recent force-pushes add comments as discussed and squash. |
/lgtm /hold cancel |
@@ -1,5 +1,5 @@ | |||
/* | |||
Copyright 2019 The Kubernetes Authors. | |||
Copyright 2021 The Kubernetes Authors. |
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.
nit: feel free to ignore, but we shouldn't be changing dates in case of moving files around
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.
This one is not just a move, there are substantial edits here.
2341f3d
to
17f9c51
Compare
The force-push to 17f9c51ff85 updated the comments on EventFunc, EventClock to remove information that is specific to FakeEventClock, and added information to the comment on FakeEventClock. |
17f9c51
to
c177aff
Compare
And the force-push to c177aff5be1 made another revision to the comment on FakeEventClock. |
/lgtm |
@MikeSpreitzer - can you please rebase? |
@MikeSpreitzer needs rebase |
So we can move off of the apimachinery clock package. Switch queueset to new clocks. Removed event clocks based on apimachinery clocks, because this PR introduces ones based on k8s.io/utils/clock . Removed interface that is implemented by only one interesting type. Simplify RealEventClock::EventAfterTime.
c177aff
to
dcb298c
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: MikeSpreitzer, 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 |
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
This PR makes an alternate version of the event clock stuff from
staging/src/k8s.io/apiserver/pkg/util/flowcontrol/fairqueuing/testing/clock
that is better in two ways:The latter is helpful so that we can make the queueset implementation take an action after a time (for duration padding).
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: