From 890eb63dccae3b945b3dea442a2f5c72f76f029f Mon Sep 17 00:00:00 2001 From: Aaron Clawson Date: Thu, 18 Nov 2021 14:51:49 +0000 Subject: [PATCH] Removed unused func, added doc strings --- debug/debug.go | 10 ---------- internal/debug/debug.go | 12 ++++++++++++ 2 files changed, 12 insertions(+), 10 deletions(-) diff --git a/debug/debug.go b/debug/debug.go index 6b5549c9c..91f488654 100644 --- a/debug/debug.go +++ b/debug/debug.go @@ -15,11 +15,7 @@ package debug // import "go.opentelemetry.io/otel/debug" import ( - "log" - "os" - "github.com/go-logr/logr" - "github.com/go-logr/stdr" "go.opentelemetry.io/otel/internal/debug" ) @@ -28,9 +24,3 @@ import ( func SetLogger(logger logr.Logger) { debug.Log = logger } - -// SetDefaultLogger configures the internal logger to use stderr and show verbose logging messages. -func SetDefaultLogger() { - SetLogger(stdr.New(log.New(os.Stderr, "", log.LstdFlags|log.Lshortfile))) - stdr.SetVerbosity(5) -} diff --git a/internal/debug/debug.go b/internal/debug/debug.go index a26942c69..e6d17142f 100644 --- a/internal/debug/debug.go +++ b/internal/debug/debug.go @@ -22,20 +22,32 @@ import ( "github.com/go-logr/stdr" ) +// globalLogger is the logging interface used within the otel api and sdk provide deatails of the internals. +// +// The default logger uses stdr which is backed by the standard `log.Logger` +// interface. This logger will only show messages at the Error Level. var globalLoggger logr.Logger = stdr.New(log.New(os.Stderr, "", log.LstdFlags|log.Lshortfile)) +// SetLogger overrides the globalLogger with l. +// +// To see Info messages use a logger with `l.V(1).Enabled() == true` +// To see Debug messages use a logger with `l.V(5).Enabled() == true` func SetLogger(l logr.Logger) { globalLoggger = l } +// Info prints messages about the general state of the API or SDK. +// This should usually be less then 5 messages a minute func Info(msg string, keysAndValues ...interface{}) { globalLoggger.V(1).Info(msg, keysAndValues...) } +// Error prints messages about exceptional states of the API or SDK. func Error(err error, msg string, keysAndValues ...interface{}) { globalLoggger.Error(err, msg, keysAndValues...) } +// Debug prints messages about all internal changes in the API or SDK. func Debug(msg string, keysAndValues ...interface{}) { globalLoggger.V(5).Info(msg, keysAndValues...) }