set loggers in 'up local' and test code

Author: estroz

commands/operator-sdk/cmd/up/local.go

@@ -38,6 +38,7 @@ import (
 	"github.com/spf13/cobra"
 	"sigs.k8s.io/controller-runtime/pkg/client/config"
 	"sigs.k8s.io/controller-runtime/pkg/manager"
+	logf "sigs.k8s.io/controller-runtime/pkg/runtime/log"
 )
 
 func NewLocalCmd() *cobra.Command {
@@ -131,6 +132,9 @@ func upLocal() {
 func upLocalAnsible() {
 	// Set the kubeconfig that the manager will be able to grab
 	os.Setenv(k8sutil.KubeConfigEnvVar, kubeConfig)
+
+	logf.SetLogger(logf.ZapLogger(false))
+
 	mgr, err := manager.New(config.GetConfigOrDie(), manager.Options{Namespace: namespace})
 	if err != nil {
 		log.Fatal(err)

pkg/logutil/log_util.go

@@ -22,13 +22,17 @@ import (
 // Usage: log.V(LogVerbosity).Info(...)
 type LogVerbosity = int
 
+// The default log verbosity, given to the most important log statements,
+// is 0: log.V(0).Info(...) == log.Info(...)
+//
+// HACK: zapr doesn't calculate log levels correctly, so we must set log
+// level integers incorrectly (according to logr) for now so they map to zapr's.
 const (
-	// Default of 0. log.V(0).Info(...) == log.Info(...)
-	_ LogVerbosity = iota
+	// VDebug == -1. log.V(VDebug).Info(...) == log.Debug(...)
+	VDebug LogVerbosity = iota - 1
+	_
 	// VWarn == 1. log.V(VWarn).Info(...) == log.Warn(...)
 	VWarn
-	// VDebug == 2. log.V(VDebug).Info(...) == log.Debug(...)
-	VDebug
 )
 
 // SDKLog is the parent logger for logs generated by operator code.

test/ansible-operator/cmd/ansible-operator/main.go

@@ -27,6 +27,7 @@ import (
 	log "github.com/sirupsen/logrus"
 	"sigs.k8s.io/controller-runtime/pkg/client/config"
 	"sigs.k8s.io/controller-runtime/pkg/manager"
+	logf "sigs.k8s.io/controller-runtime/pkg/runtime/log"
 )
 
 func printVersion() {
@@ -38,6 +39,8 @@ func printVersion() {
 func main() {
 	flag.Parse()
 
+	logf.SetLogger(logf.ZapLogger(true))
+
 	namespace, err := k8sutil.GetWatchNamespace()
 	if err != nil {
 		log.Fatalf("failed to get watch namespace: (%v)", err)