reduce logging (gets it the way of tests)

Author: zeitlinger

testing-common/src/main/java/io/opentelemetry/instrumentation/testing/provider/TestBridgeLogRecordExporterComponentProvider.java

@@ -8,15 +8,10 @@
 import io.opentelemetry.api.incubator.config.DeclarativeConfigProperties;
 import io.opentelemetry.sdk.autoconfigure.spi.internal.ComponentProvider;
 import io.opentelemetry.sdk.logs.export.LogRecordExporter;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 public class TestBridgeLogRecordExporterComponentProvider
     implements ComponentProvider<LogRecordExporter> {
 
-  private static final Logger logger =
-      LoggerFactory.getLogger(TestBridgeLogRecordExporterComponentProvider.class);
-
   private static LogRecordExporter logRecordExporter;
 
   @Override
@@ -35,9 +30,6 @@ public LogRecordExporter create(DeclarativeConfigProperties config) {
   }
 
   public static void setLogRecordExporter(LogRecordExporter logRecordExporter) {
-    logger.info(
-        "Setting TestLogRecordExporterComponentProvider logRecord exporter to {}",
-        logRecordExporter.getClass().getName());
     TestBridgeLogRecordExporterComponentProvider.logRecordExporter = logRecordExporter;
   }
 }

testing-common/src/main/java/io/opentelemetry/instrumentation/testing/provider/TestBridgeMetricExporterComponentProvider.java

@@ -8,15 +8,10 @@
 import io.opentelemetry.api.incubator.config.DeclarativeConfigProperties;
 import io.opentelemetry.sdk.autoconfigure.spi.internal.ComponentProvider;
 import io.opentelemetry.sdk.metrics.export.MetricExporter;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 public class TestBridgeMetricExporterComponentProvider
     implements ComponentProvider<MetricExporter> {
 
-  private static final Logger logger =
-      LoggerFactory.getLogger(TestBridgeMetricExporterComponentProvider.class);
-
   private static MetricExporter metricExporter;
 
   @Override
@@ -35,9 +30,6 @@ public MetricExporter create(DeclarativeConfigProperties config) {
   }
 
   public static void setMetricExporter(MetricExporter metricExporter) {
-    logger.info(
-        "Setting TestMetricExporterComponentProvider metric exporter to {}",
-        metricExporter.getClass().getName());
     TestBridgeMetricExporterComponentProvider.metricExporter = metricExporter;
   }
 }

testing-common/src/main/java/io/opentelemetry/instrumentation/testing/provider/TestBridgeSpanExporterComponentProvider.java

@@ -8,14 +8,9 @@
 import io.opentelemetry.api.incubator.config.DeclarativeConfigProperties;
 import io.opentelemetry.sdk.autoconfigure.spi.internal.ComponentProvider;
 import io.opentelemetry.sdk.trace.export.SpanExporter;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 public class TestBridgeSpanExporterComponentProvider implements ComponentProvider<SpanExporter> {
 
-  private static final Logger logger =
-      LoggerFactory.getLogger(TestBridgeSpanExporterComponentProvider.class);
-
   private static SpanExporter spanExporter;
 
   @Override
@@ -34,9 +29,6 @@ public SpanExporter create(DeclarativeConfigProperties config) {
   }
 
   public static void setSpanExporter(SpanExporter spanExporter) {
-    logger.info(
-        "Setting TestSpanExporterComponentProvider span exporter to {}",
-        spanExporter.getClass().getName());
     TestBridgeSpanExporterComponentProvider.spanExporter = spanExporter;
   }
 }

testing/agent-exporter/src/main/java/io/opentelemetry/javaagent/testing/provider/TestBridgeLogRecordExporterComponentProvider.java

@@ -8,15 +8,10 @@
 import io.opentelemetry.api.incubator.config.DeclarativeConfigProperties;
 import io.opentelemetry.sdk.autoconfigure.spi.internal.ComponentProvider;
 import io.opentelemetry.sdk.logs.export.LogRecordExporter;
-import java.util.logging.Level;
-import java.util.logging.Logger;
 
 public class TestBridgeLogRecordExporterComponentProvider
     implements ComponentProvider<LogRecordExporter> {
 
-  private static final Logger logger =
-      Logger.getLogger(TestBridgeLogRecordExporterComponentProvider.class.getName());
-
   private static LogRecordExporter logRecordExporter;
 
   @Override
@@ -35,8 +30,6 @@ public LogRecordExporter create(DeclarativeConfigProperties config) {
   }
 
   public static void setLogRecordExporter(LogRecordExporter logRecordExporter) {
-    logger.log(
-        Level.INFO, "Setting logRecord exporter to {0}", logRecordExporter.getClass().getName());
     TestBridgeLogRecordExporterComponentProvider.logRecordExporter = logRecordExporter;
   }
 }

testing/agent-exporter/src/main/java/io/opentelemetry/javaagent/testing/provider/TestBridgeMetricExporterComponentProvider.java

@@ -8,15 +8,10 @@
 import io.opentelemetry.api.incubator.config.DeclarativeConfigProperties;
 import io.opentelemetry.sdk.autoconfigure.spi.internal.ComponentProvider;
 import io.opentelemetry.sdk.metrics.export.MetricExporter;
-import java.util.logging.Level;
-import java.util.logging.Logger;
 
 public class TestBridgeMetricExporterComponentProvider
     implements ComponentProvider<MetricExporter> {
 
-  private static final Logger logger =
-      Logger.getLogger(TestBridgeMetricExporterComponentProvider.class.getName());
-
   private static MetricExporter metricExporter;
 
   @Override
@@ -35,8 +30,6 @@ public MetricExporter create(DeclarativeConfigProperties config) {
   }
 
   public static void setMetricExporter(MetricExporter metricExporter) {
-    logger.log(
-        Level.INFO, "Setting logRecord exporter to {0}", metricExporter.getClass().getName());
     TestBridgeMetricExporterComponentProvider.metricExporter = metricExporter;
   }
 }

testing/agent-exporter/src/main/java/io/opentelemetry/javaagent/testing/provider/TestBridgeSpanExporterComponentProvider.java

@@ -8,14 +8,9 @@
 import io.opentelemetry.api.incubator.config.DeclarativeConfigProperties;
 import io.opentelemetry.sdk.autoconfigure.spi.internal.ComponentProvider;
 import io.opentelemetry.sdk.trace.export.SpanExporter;
-import java.util.logging.Level;
-import java.util.logging.Logger;
 
 public class TestBridgeSpanExporterComponentProvider implements ComponentProvider<SpanExporter> {
 
-  private static final Logger logger =
-      Logger.getLogger(TestBridgeSpanExporterComponentProvider.class.getName());
-
   private static SpanExporter spanExporter;
 
   @Override
@@ -34,7 +29,6 @@ public SpanExporter create(DeclarativeConfigProperties config) {
   }
 
   public static void setSpanExporter(SpanExporter spanExporter) {
-    logger.log(Level.INFO, "Setting span exporter to {0}", spanExporter.getClass().getName());
     TestBridgeSpanExporterComponentProvider.spanExporter = spanExporter;
   }
 }