From 2b56144c502b64678c8e4ef569e37d63071119f7 Mon Sep 17 00:00:00 2001
From: jkhelil <jkhelil@redhat.com>
Date: Thu, 8 Jul 2021 17:41:07 +0200
Subject: [PATCH] use ctxlog from shipwright packages

---
 cmd/operator/main.go      | 21 +++++++++++----------
 controllers/suite_test.go |  1 -
 2 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/cmd/operator/main.go b/cmd/operator/main.go
index af4c5953..8d7dc933 100644
--- a/cmd/operator/main.go
+++ b/cmd/operator/main.go
@@ -12,6 +12,7 @@ import (
 	// to ensure that exec-entrypoint and run can make use of them.
 	_ "k8s.io/client-go/plugin/pkg/client/auth"
 
+	"github.com/shipwright-io/build/pkg/ctxlog"
 	apiextv1 "k8s.io/apiextensions-apiserver/pkg/apis/apiextensions/v1"
 	"k8s.io/apimachinery/pkg/runtime"
 	utilruntime "k8s.io/apimachinery/pkg/util/runtime"
@@ -26,8 +27,7 @@ import (
 )
 
 var (
-	scheme   = runtime.NewScheme()
-	setupLog = ctrl.Log.WithName("setup")
+	scheme = runtime.NewScheme()
 )
 
 // variables to hold command-line flags
@@ -64,7 +64,9 @@ func main() {
 	opts.BindFlags(flag.CommandLine)
 	flag.Parse()
 
-	ctrl.SetLogger(zap.New(zap.UseFlagOptions(&opts)))
+	l := ctxlog.NewLogger("shipwrightBuild-controller")
+
+	ctx := ctxlog.NewParentContext(l)
 
 	mgr, err := ctrl.NewManager(ctrl.GetConfigOrDie(), ctrl.Options{
 		Scheme:                 scheme,
@@ -75,32 +77,31 @@ func main() {
 		LeaderElectionID:       "01a9b2d1.shipwright.io",
 	})
 	if err != nil {
-		setupLog.Error(err, "unable to start manager")
+		ctxlog.Error(ctx, err, "unable to start manager")
 		os.Exit(1)
 	}
 
 	if err = (&controllers.ShipwrightBuildReconciler{
 		Client: mgr.GetClient(),
-		Log:    ctrl.Log.WithName("controllers").WithName("ShipwrightBuild"),
 		Scheme: mgr.GetScheme(),
 	}).SetupWithManager(mgr); err != nil {
-		setupLog.Error(err, "unable to create controller", "controller", "ShipwrightBuild")
+		ctxlog.Error(ctx, err, "unable to create controller")
 		os.Exit(1)
 	}
 	// +kubebuilder:scaffold:builder
 
 	if err := mgr.AddHealthzCheck("health", healthz.Ping); err != nil {
-		setupLog.Error(err, "unable to set up health check")
+		ctxlog.Error(ctx, err, "unable to set up health check")
 		os.Exit(1)
 	}
 	if err := mgr.AddReadyzCheck("check", healthz.Ping); err != nil {
-		setupLog.Error(err, "unable to set up ready check")
+		ctxlog.Error(ctx, err, "unable to set up ready check")
 		os.Exit(1)
 	}
 
-	setupLog.Info("starting manager")
+	ctxlog.Info(ctx, "starting manager")
 	if err := mgr.Start(ctrl.SetupSignalHandler()); err != nil {
-		setupLog.Error(err, "problem running manager")
+		ctxlog.Error(ctx, err, "problem running manager")
 		os.Exit(1)
 	}
 }
diff --git a/controllers/suite_test.go b/controllers/suite_test.go
index 81cb381c..07c0c88d 100644
--- a/controllers/suite_test.go
+++ b/controllers/suite_test.go
@@ -73,7 +73,6 @@ var _ = BeforeSuite(func() {
 	err = (&ShipwrightBuildReconciler{
 		Client: mgr.GetClient(),
 		Scheme: scheme.Scheme,
-		Log:    ctrl.Log.WithName("controllers").WithName("shipwrightbuild"),
 	}).SetupWithManager(mgr)
 	Expect(err).NotTo(HaveOccurred())