• Kubernetes Submit Queue's avatar
    Merge pull request #41261 from ncdc/shared-informers-07-resourcequota · 1ad5cea2
    Kubernetes Submit Queue authored
    Automatic merge from submit-queue
    
    Switch resourcequota controller to shared informers
    
    Originally part of #40097 
    
    I have had some issues with this change in the past, when I updated `pkg/quota` to use the new informers while `pkg/controller/resourcequota` remained on the old informers. In this PR, both are switched to using the new informers. The issues in the past were lots of flakey test failures in the ResourceQuota e2es, where it would randomly fail to see deletions and handle replenishment. I am hoping that now that everything here is consistently using the new informers, there won't be any more of these flakes, but it's something to keep an eye out for.
    
    I also think `pkg/controller/resourcequota` could be cleaned up. I don't think there's really any need for `replenishment_controller.go` any more since it's no longer running individual controllers per kind to replenish. It instead just uses the shared informer and adds event handlers to it. But maybe we do that in a follow up.
    
    cc @derekwaynecarr @smarterclayton @wojtek-t @deads2k @sttts @liggitt @timothysc @kubernetes/sig-scalability-pr-reviews
    1ad5cea2
Name
Last commit
Last update
..
admission Loading commit data...
auth Loading commit data...
scheduler Loading commit data...