pkg/log/zap: clarify zap level vs. logr verbosity with link to zapr e…

…xplanation and test cases

Signed-off-by: Eric Stroczynski <ericstroczynski@gmail.com>

pkg/log/zap/zap.go

@@ -101,8 +101,9 @@ func newConsoleEncoder(opts ...EncoderConfigOption) zapcore.Encoder {
 	return zapcore.NewConsoleEncoder(encoderConfig)
 }
 
-// Level sets the the minimum enabled logging level e.g Debug, Info
-// See Options.Level
+// Level sets Options.Level, which configures the the minimum enabled logging level e.g Debug, Info.
+// If using zapcore.Level(N), where N > 0, set zapcore.Level(-1*N) instead.
+// See https://pkg.go.dev/github.com/go-logr/zapr for how zap level relates to logr verbosity.
 func Level(level zapcore.LevelEnabler) func(o *Options) {
 	return func(o *Options) {
 		o.Level = level
@@ -151,8 +152,9 @@ type Options struct {
 	//
 	// Deprecated: Use DestWriter instead
 	DestWritter io.Writer
-	// Level configures the verbosity of the logging.  Defaults to Debug when
-	// Development is true and Info otherwise
+	// Level configures the verbosity of the logging.
+	// Defaults to Debug when Development is true and Info otherwise.
+	// See https://pkg.go.dev/github.com/go-logr/zapr for how zap level relates to logr verbosity.
 	Level zapcore.LevelEnabler
 	// StacktraceLevel is the level at and above which stacktraces will
 	// be recorded for all messages. Defaults to Warn when Development

pkg/log/zap/zap_test.go

@@ -472,7 +472,7 @@ var _ = Describe("Zap log level flag options setup", func() {
 		})
 	})
 
-	Context("with encoder options provided programmatically.", func() {
+	Context("with encoder options provided programmatically", func() {
 
 		It("Should set Console Encoder, with given Nanos TimeEncoder option.", func() {
 			logOut := new(bytes.Buffer)
@@ -517,5 +517,32 @@ var _ = Describe("Zap log level flag options setup", func() {
 			Expect(string(outRaw)).Should(ContainSubstring("MillisTimeFormat"))
 		})
 
+		Context("using Level()", func() {
+			var logOut *bytes.Buffer
+
+			BeforeEach(func() {
+				logOut = new(bytes.Buffer)
+			})
+
+			It("logs with negative logr level", func() {
+				By("setting up the logger")
+				logger := New(WriteTo(logOut), Level(zapcore.Level(-3)))
+				logger.V(3).Info("test 3") // Should be logged
+				Expect(string(logOut.Bytes())).To(ContainSubstring(`"msg":"test 3"`))
+				logOut.Truncate(0)
+				logger.V(4).Info("test 4") // Should not be logged
+				Expect(string(logOut.Bytes())).To(BeEmpty())
+				logger.V(-3).Info("test -3") // Log a panic, since V(-1*N) for all N > 0 is not permitted.
+				Expect(string(logOut.Bytes())).To(ContainSubstring(`"level":"dpanic"`))
+			})
+			It("does not log with positive logr level", func() {
+				By("setting up the logger")
+				logger := New(WriteTo(logOut), Level(zapcore.Level(1)))
+				logger.V(1).Info("test 1")
+				Expect(string(logOut.Bytes())).To(BeEmpty())
+				logger.V(3).Info("test 3")
+				Expect(string(logOut.Bytes())).To(BeEmpty())
+			})
+		})
 	})
 })