Remove the exporter Debug configuration flag and just use the informa…

…tion available in the zap.Logger
open-telemetry · Dec 2, 2019 · 3493c62 · 3493c62
1 parent 3bdc21a
commit 3493c62
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/exporter/azuremonitorexporter/config.go b/exporter/azuremonitorexporter/config.go
@@ -26,5 +26,4 @@ type Config struct {
 	InstrumentationKey            string `mapstructure:"instrumentation_key"`
 	MaxBatchSize                  int    `mapstructure:"maxbatchsize"`
 	MaxBatchIntervalInSeconds     int    `mapstructure:"maxbatchinterval_sec"`
-	Debug                         bool   `mapstructure:"debug"`
 }
diff --git a/exporter/azuremonitorexporter/factory.go b/exporter/azuremonitorexporter/factory.go
@@ -94,7 +94,8 @@ func (f *Factory) getTransportChannel(exporterConfig *Config, logger *zap.Logger
 
 		f.TransportChannel = telemetryClient.Channel()
 
-		if exporterConfig.Debug {
+		// Don't even bother enabling the AppInsights diagnostics listener unless debug logging is enabled
+		if checkedEntry := logger.Check(zap.DebugLevel, ""); checkedEntry != nil {
 			appinsights.NewDiagnosticsMessageListener(func(msg string) error {
 				logger.Debug(msg)
 				return nil