From bbf5af22c18dad12247595cf2d9be5f5151ac76c Mon Sep 17 00:00:00 2001 From: Brian Goff Date: Wed, 12 Jun 2019 13:42:03 -0700 Subject: [PATCH] Remove resourcemanager from conroller public API (#664) We still use it internally, but this does not need to be part of the public API. Instead just have callers pass us the relevent listers and we create our own resource manager. --- virtual-kubelet/commands/root/root.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/virtual-kubelet/commands/root/root.go b/virtual-kubelet/commands/root/root.go index cafbcb8..037cb34 100644 --- a/virtual-kubelet/commands/root/root.go +++ b/virtual-kubelet/commands/root/root.go @@ -181,7 +181,9 @@ func runRootCommand(ctx context.Context, c Opts) error { PodInformer: podInformer, EventRecorder: eb.NewRecorder(scheme.Scheme, corev1.EventSource{Component: path.Join(pNode.Name, "pod-controller")}), Provider: p, - ResourceManager: rm, + SecretLister: secretInformer.Lister(), + ConfigMapLister: configMapInformer.Lister(), + ServiceLister: serviceInformer.Lister(), }) if err != nil { return errors.Wrap(err, "error setting up pod controller")