Skip to content

Commit

Permalink
No need to start the informers
Browse files Browse the repository at this point in the history
  • Loading branch information
tcnghia committed Mar 16, 2022
1 parent 544cc6b commit 860b938
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions pkg/reconciler/clusterimagepolicy/controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -51,13 +51,6 @@ func NewController(
secretInformer := secretinformer.Get(ctx)
configMapInformer := cminformer.Get(ctx)

// Start the informers we got from the SharedInformerFactory above because
// injection doesn't do that for us since we're injecting the Factory and
// not the informers.
if err := controller.StartInformers(ctx.Done(), secretInformer.Informer(), configMapInformer.Informer()); err != nil {
logging.FromContext(ctx).Fatalf("Failed to start informers: %w", err)
}

r := &Reconciler{
secretlister: secretInformer.Lister(),
configmaplister: configMapInformer.Lister(),
Expand Down

0 comments on commit 860b938

Please sign in to comment.