🏃 Remove unsed Config attribute from controller

The controller contains the unused Config attribute. Since its exported,
linters will not complain about it, but since its in an internal
package, it is not part of our public api.

Author: alvaroaleman

pkg/controller/controller.go

@@ -104,7 +104,6 @@ func NewUnmanaged(name string, mgr manager.Manager, options Options) (Controller
 	c := &controller.Controller{
 		Do:       options.Reconciler,
 		Cache:    mgr.GetCache(),
-		Config:   mgr.GetConfig(),
 		Scheme:   mgr.GetScheme(),
 		Client:   mgr.GetClient(),
 		Recorder: mgr.GetEventRecorderFor(name),

pkg/internal/controller/controller.go

@@ -24,7 +24,6 @@ import (
 	"k8s.io/apimachinery/pkg/runtime"
 	utilruntime "k8s.io/apimachinery/pkg/util/runtime"
 	"k8s.io/apimachinery/pkg/util/wait"
-	"k8s.io/client-go/rest"
 	"k8s.io/client-go/tools/record"
 	"k8s.io/client-go/util/workqueue"
 	"sigs.k8s.io/controller-runtime/pkg/cache"
@@ -64,10 +63,6 @@ type Controller struct {
 	// informers are injected by the controllerManager when controllerManager.Start is called
 	Cache cache.Cache
 
-	// Config is the rest.Config used to talk to the apiserver.  Defaults to one of in-cluster, environment variable
-	// specified, or the ~/.kube/Config.
-	Config *rest.Config
-
 	// MakeQueue constructs the queue for this controller once the controller is ready to start.
 	// This exists because the standard Kubernetes workqueues start themselves immediately, which
 	// leads to goroutine leaks if something calls controller.New repeatedly.