Use otlp collector for logs

Author: kannanjgithub

interop-testing/src/main/java/io/grpc/testing/integration/XdsTestClient.java

@@ -269,10 +269,13 @@ private void run() {
           .enableTracing()
           .sdk(AutoConfiguredOpenTelemetrySdk.builder()
               .addPropertiesSupplier(() -> ImmutableMap.of(
-                  "otel.logs.exporter", "otlp",
                   "otel.metrics.exporter", "prometheus",
+                  "otel.logs.exporter", "otlp",
                   "otel.traces.exporter", "otlp",
+                  // Only handles traces
                   "otel.exporter.otlp.endpoint", "https://telemetry.googleapis.com",
+                  // Set logs endpoint to the OTLP default Collector
+                  "otel.exporter.otlp.logs.endpoint", "http://localhost:4317",
                   "otel.exporter.otlp.protocol", "http/protobuf"))
               .build()
               .getOpenTelemetrySdk())

opentelemetry/src/main/java/io/grpc/opentelemetry/OpenTelemetryTracingModule.java

@@ -106,9 +106,9 @@ final class OpenTelemetryTracingModule {
         .setInstrumentationVersion(IMPLEMENTATION_VERSION)
         .build();
     this.contextPropagators = checkNotNull(openTelemetry.getPropagators(), "contextPropagators");
-    this.otelLogger = openTelemetry.getLogsBridge().loggerBuilder("grpc-java-client-scope")
-        .setInstrumentationVersion("1.0.0")
-        .build();
+
+    // 3. Get a Logger from the Bridge API
+    this.otelLogger = openTelemetry.getLogsBridge().get("io.grpc.opentelemetry");
   }
 
   @VisibleForTesting
@@ -121,7 +121,7 @@ Tracer getTracer() {
    */
   @VisibleForTesting
   CallAttemptsTracerFactory newClientCallTracer(Span clientSpan, MethodDescriptor<?, ?> method) {
-    return new CallAttemptsTracerFactory(clientSpan, method, otelLogger);
+    return new CallAttemptsTracerFactory(clientSpan, method);
   }
 
   /**
@@ -147,14 +147,11 @@ final class CallAttemptsTracerFactory extends ClientStreamTracer.Factory {
     volatile int callEnded;
     private final Span clientSpan;
     private final String fullMethodName;
-    private final io.opentelemetry.api.logs.Logger otelLogger;
 
-    CallAttemptsTracerFactory(Span clientSpan, MethodDescriptor<?, ?> method,
-        io.opentelemetry.api.logs.Logger otelLogger) {
+    CallAttemptsTracerFactory(Span clientSpan, MethodDescriptor<?, ?> method) {
       checkNotNull(method, "method");
       this.fullMethodName = checkNotNull(method.getFullMethodName(), "fullMethodName");
       this.clientSpan = checkNotNull(clientSpan, "clientSpan");
-      this.otelLogger = otelLogger;
     }
 
     @Override