Merge pull request #70072 from yue9944882/chore/cleanup-serviceaccount-controller-options

Fixes wild serviceaccount controller resync options

Merge pull request #70072 from yue9944882/chore/cleanup-serviceaccount-controller-options
dbdcb7e0 · k8s-ci-robot · GitHub · 15c15bed · 4624c131 · dbdcb7e0
Unverified Commit dbdcb7e0 authored Oct 22, 2018 by k8s-ci-robot Committed by GitHub Oct 22, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 4 deletions

serviceaccounts_controller.go pkg/controller/serviceaccount/serviceaccounts_controller.go +4 -4

No files found.
--- a/pkg/controller/serviceaccount/serviceaccounts_controller.go
+++ b/pkg/controller/serviceaccount/serviceaccounts_controller.go
@@ -73,16 +73,16 @@ func NewServiceAccountsController(saInformer coreinformers.ServiceAccountInforme
 		}
 	}

-	saInformer.Informer().AddEventHandler(cache.ResourceEventHandlerFuncs{
+	saInformer.Informer().AddEventHandlerWithResyncPeriod(cache.ResourceEventHandlerFuncs{
 		DeleteFunc: e.serviceAccountDeleted,
-	})
+	}, options.ServiceAccountResync)
 	e.saLister = saInformer.Lister()
 	e.saListerSynced = saInformer.Informer().HasSynced

-	nsInformer.Informer().AddEventHandler(cache.ResourceEventHandlerFuncs{
+	nsInformer.Informer().AddEventHandlerWithResyncPeriod(cache.ResourceEventHandlerFuncs{
 		AddFunc:    e.namespaceAdded,
 		UpdateFunc: e.namespaceUpdated,
-	})
+	}, options.NamespaceResync)
 	e.nsLister = nsInformer.Lister()
 	e.nsListerSynced = nsInformer.Informer().HasSynced