Merge pull request #1806 from sbueringer/pr-controller-name-kv

🌱 Add controller-name as kv pair to logger instead of as name
kubernetes-sigs · Feb 15, 2022 · 5b2eaae · 5b2eaae
2 parents b5bb688 + b4d1c56
commit 5b2eaae
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/pkg/controller/controller.go b/pkg/controller/controller.go
@@ -23,6 +23,7 @@ import (
 
 	"github.com/go-logr/logr"
 	"k8s.io/client-go/util/workqueue"
+
 	"sigs.k8s.io/controller-runtime/pkg/handler"
 	"sigs.k8s.io/controller-runtime/pkg/internal/controller"
 	"sigs.k8s.io/controller-runtime/pkg/manager"
@@ -135,7 +136,7 @@ func NewUnmanaged(name string, mgr manager.Manager, options Options) (Controller
 		CacheSyncTimeout:        options.CacheSyncTimeout,
 		SetFields:               mgr.SetFields,
 		Name:                    name,
-		Log:                     options.Log.WithName("controller").WithName(name),
+		Log:                     options.Log.WithName("controller").WithName(name).WithValues("controller", name),
 		RecoverPanic:            options.RecoverPanic,
 	}, nil
 }