update TODO's/comments

Author: cbarbian-sap

internal/cluster/factory.go

@@ -147,6 +147,7 @@ func (f *ClientFactory) Get(kubeConfig []byte, impersonationUser string, imperso
 	f.clients[key] = client
 
 	// TODO: add some (debug) log output when new client is created; unfortunately, we have no logger in here ...
+	// maybe we could (at least in Get()) get one from the reconcile context ...
 	// TODO: add metrics about running clients
 	return client, nil
 }

internal/cluster/types.go

@@ -12,6 +12,7 @@ import (
 )
 
 // TODO: the Client interface should be public, because it is returned by component.GetClientFromContext()
+// TODO: should we add a Config() method, i.e. expose the rest.Config used by the client?
 
 // The Client interface extends the controller-runtime client by discovery and event recording capabilities.
 type Client interface {