wip(refactor): make log pkg adhere to composition pattern

Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>

Author: armru

pkg/log/flags.go

@@ -21,6 +21,7 @@ import (
 	"fmt"
 	"os"
 
+	"github.com/go-logr/logr"
 	"github.com/spf13/pflag"
 	"go.uber.org/zap/zapcore"
 	"k8s.io/klog/v2"
@@ -80,15 +81,12 @@ func GetFieldsRemapFlags() (res []string) {
 	return res
 }
 
-// ConfigureLogging configure the logging honoring the flags
-// passed from the user
-// This is executed after args were already parsed.
-func (l *Flags) ConfigureLogging() {
+func (l *Flags) BuildLogger() logr.Logger {
 	if l.zapOptions.TimeEncoder == nil {
 		l.zapOptions.TimeEncoder = zapcore.RFC3339NanoTimeEncoder
 	}
 
-	logger := zap.New(zap.UseFlagOptions(&l.zapOptions), customLevel, customDestination, remapKeys)
+	lgr := zap.New(zap.UseFlagOptions(&l.zapOptions), customLevel, customDestination, remapKeys)
 	switch logLevel {
 	case ErrorLevelString,
 		WarningLevelString,
@@ -97,13 +95,26 @@ func (l *Flags) ConfigureLogging() {
 		TraceLevelString:
 		break
 	default:
-		logger.Info("Invalid log level, defaulting", "level", logLevel, "default", DefaultLevel)
+		lgr.Info("Invalid log level, defaulting", "level", logLevel, "default", DefaultLevel)
 	}
 
-	redirectStdLog(logger)
-	controllerruntime.SetLogger(logger)
-	klog.SetLogger(logger)
-	SetLogger(logger)
+	return lgr
+}
+
+// ConfigureLogging configure the logging honoring the flags
+// passed from the user
+// This is executed after args were already parsed.
+func (l *Flags) ConfigureLogging() {
+	lgr := l.BuildLogger()
+	SetGlobalLogger(lgr)
+}
+
+// SetGlobalLogger configure the underlying loggers
+func SetGlobalLogger(lgr logr.Logger) {
+	redirectStdLog(lgr)
+	controllerruntime.SetLogger(lgr)
+	klog.SetLogger(lgr)
+	SetLogger(lgr)
 }
 
 func getLogLevel(l string) zapcore.Level {