From 40160ad36f0e9e8c95e399b897cbd6a15b5e0c92 Mon Sep 17 00:00:00 2001 From: Yash Desai Date: Wed, 3 Apr 2019 11:19:30 -0700 Subject: [PATCH] Resource manager: add service lister and remove unused lock. (#559) * Remove unused lock from the resource manager. * Add service lister to the resource manager. This change adds a service lister in the resource manager. This will be used to set the service env vars. Also added a List method to the resource manager and a simple test to confirm it's a pass through. --- root.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/root.go b/root.go index 4e10a0f..cac569f 100644 --- a/root.go +++ b/root.go @@ -312,8 +312,11 @@ func initConfig() { secretInformer := scmInformerFactory.Core().V1().Secrets() configMapInformer := scmInformerFactory.Core().V1().ConfigMaps() + // Create a service informer so we can pass its lister to the resource manager. + serviceInformer := scmInformerFactory.Core().V1().Services() + // Create a new instance of the resource manager that uses the listers above for pods, secrets and config maps. - rm, err = manager.NewResourceManager(podInformer.Lister(), secretInformer.Lister(), configMapInformer.Lister()) + rm, err = manager.NewResourceManager(podInformer.Lister(), secretInformer.Lister(), configMapInformer.Lister(), serviceInformer.Lister()) if err != nil { logger.WithError(err).Fatal("Error initializing resource manager") }