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
use a separate queue for initial quota calculation #29133
Conversation
@@ -63,6 +63,8 @@ type ResourceQuotaController struct { | |||
rqController *framework.Controller | |||
// ResourceQuota objects that need to be synchronized | |||
queue workqueue.RateLimitingInterface | |||
// missingStatusQueue holds objects that are missing the initial usage informatino |
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: update godoc to reflect name (suspect you changed it at some point)
comments addressed. |
LGTM |
GCE e2e build/test passed for commit 2ea3422. |
@@ -63,6 +63,8 @@ type ResourceQuotaController struct { | |||
rqController *framework.Controller | |||
// ResourceQuota objects that need to be synchronized | |||
queue workqueue.RateLimitingInterface | |||
// missingUsageQueue holds objects that are missing the initial usage informatino |
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.
typo: information
@k8s-bot test this [submit-queue is verifying that this PR is safe to merge] |
GCE e2e build/test passed for commit 2ea3422. |
Automatic merge from submit-queue |
When the quota controller gets backed up on resyncs, it can take a long time to observe the first usage stats which are needed by the admission plugin. This creates a second queue to prioritize the initial calculation.