Copy component loader testing (#7596)

trask · web-flow · commit 20f0a3a9aab2 · 2025-08-19T13:05:12.000-07:00
diff --git a/exporters/common/src/main/java/io/opentelemetry/exporter/internal/grpc/GrpcExporterBuilder.java b/exporters/common/src/main/java/io/opentelemetry/exporter/internal/grpc/GrpcExporterBuilder.java
@@ -205,6 +205,7 @@ public GrpcExporterBuilder<T> copy() {
     copy.meterProviderSupplier = meterProviderSupplier;
     copy.internalTelemetryVersion = internalTelemetryVersion;
     copy.grpcChannel = grpcChannel;
+    copy.componentLoader = componentLoader;
     return copy;
   }
 
diff --git a/exporters/common/src/main/java/io/opentelemetry/exporter/internal/http/HttpExporterBuilder.java b/exporters/common/src/main/java/io/opentelemetry/exporter/internal/http/HttpExporterBuilder.java
@@ -204,6 +204,7 @@ public HttpExporterBuilder<T> copy() {
     copy.meterProviderSupplier = meterProviderSupplier;
     copy.internalTelemetryVersion = internalTelemetryVersion;
     copy.proxyOptions = proxyOptions;
+    copy.componentLoader = componentLoader;
     return copy;
   }
 
diff --git a/exporters/otlp/testing-internal/src/main/java/io/opentelemetry/exporter/otlp/testing/internal/AbstractHttpTelemetryExporterTest.java b/exporters/otlp/testing-internal/src/main/java/io/opentelemetry/exporter/otlp/testing/internal/AbstractHttpTelemetryExporterTest.java
@@ -27,6 +27,7 @@
 import com.linecorp.armeria.testing.junit5.server.SelfSignedCertificateExtension;
 import com.linecorp.armeria.testing.junit5.server.ServerExtension;
 import io.github.netmikey.logunit.api.LogCapturer;
+import io.opentelemetry.common.ComponentLoader;
 import io.opentelemetry.exporter.internal.FailedExportException;
 import io.opentelemetry.exporter.internal.TlsUtil;
 import io.opentelemetry.exporter.internal.compression.GzipCompressor;
@@ -839,6 +840,7 @@ void toBuilderEquality()
                     .setInitialBackoff(Duration.ofMillis(50))
                     .setBackoffMultiplier(1.3)
                     .build())
+            .setComponentLoader(ComponentLoader.forClassLoader(new ClassLoader() {}))
             .build()) {
       Object unwrapped = exporter.unwrap();
       Field builderField = unwrapped.getClass().getDeclaredField("builder");
diff --git a/exporters/otlp/testing-internal/src/main/java/io/opentelemetry/exporter/otlp/testing/internal/GrpcLogRecordExporterBuilderWrapper.java b/exporters/otlp/testing-internal/src/main/java/io/opentelemetry/exporter/otlp/testing/internal/GrpcLogRecordExporterBuilderWrapper.java
@@ -7,6 +7,7 @@
 
 import io.grpc.ManagedChannel;
 import io.opentelemetry.api.metrics.MeterProvider;
+import io.opentelemetry.common.ComponentLoader;
 import io.opentelemetry.exporter.otlp.logs.OtlpGrpcLogRecordExporterBuilder;
 import io.opentelemetry.sdk.common.InternalTelemetryVersion;
 import io.opentelemetry.sdk.common.export.ProxyOptions;
@@ -122,6 +123,13 @@ public TelemetryExporterBuilder<LogRecordData> setServiceClassLoader(
     return this;
   }
 
+  @Override
+  public TelemetryExporterBuilder<LogRecordData> setComponentLoader(
+      ComponentLoader componentLoader) {
+    builder.setComponentLoader(componentLoader);
+    return this;
+  }
+
   @Override
   public TelemetryExporterBuilder<LogRecordData> setExecutorService(
       ExecutorService executorService) {
diff --git a/exporters/otlp/testing-internal/src/main/java/io/opentelemetry/exporter/otlp/testing/internal/GrpcMetricExporterBuilderWrapper.java b/exporters/otlp/testing-internal/src/main/java/io/opentelemetry/exporter/otlp/testing/internal/GrpcMetricExporterBuilderWrapper.java
@@ -7,6 +7,7 @@
 
 import io.grpc.ManagedChannel;
 import io.opentelemetry.api.metrics.MeterProvider;
+import io.opentelemetry.common.ComponentLoader;
 import io.opentelemetry.exporter.otlp.metrics.OtlpGrpcMetricExporterBuilder;
 import io.opentelemetry.sdk.common.InternalTelemetryVersion;
 import io.opentelemetry.sdk.common.export.ProxyOptions;
@@ -122,6 +123,12 @@ public TelemetryExporterBuilder<MetricData> setServiceClassLoader(
     return this;
   }
 
+  @Override
+  public TelemetryExporterBuilder<MetricData> setComponentLoader(ComponentLoader componentLoader) {
+    builder.setComponentLoader(componentLoader);
+    return this;
+  }
+
   @Override
   public TelemetryExporterBuilder<MetricData> setExecutorService(ExecutorService executorService) {
     builder.setExecutorService(executorService);
diff --git a/exporters/otlp/testing-internal/src/main/java/io/opentelemetry/exporter/otlp/testing/internal/GrpcProfilesExporterBuilderWrapper.java b/exporters/otlp/testing-internal/src/main/java/io/opentelemetry/exporter/otlp/testing/internal/GrpcProfilesExporterBuilderWrapper.java
@@ -7,6 +7,7 @@
 
 import io.grpc.ManagedChannel;
 import io.opentelemetry.api.metrics.MeterProvider;
+import io.opentelemetry.common.ComponentLoader;
 import io.opentelemetry.exporter.otlp.profiles.OtlpGrpcProfilesExporterBuilder;
 import io.opentelemetry.exporter.otlp.profiles.ProfileData;
 import io.opentelemetry.sdk.common.InternalTelemetryVersion;
@@ -122,6 +123,12 @@ public TelemetryExporterBuilder<ProfileData> setServiceClassLoader(
     return this;
   }
 
+  @Override
+  public TelemetryExporterBuilder<ProfileData> setComponentLoader(ComponentLoader componentLoader) {
+    builder.setComponentLoader(componentLoader);
+    return this;
+  }
+
   @Override
   public TelemetryExporterBuilder<ProfileData> setExecutorService(ExecutorService executorService) {
     builder.setExecutorService(executorService);
diff --git a/exporters/otlp/testing-internal/src/main/java/io/opentelemetry/exporter/otlp/testing/internal/GrpcSpanExporterBuilderWrapper.java b/exporters/otlp/testing-internal/src/main/java/io/opentelemetry/exporter/otlp/testing/internal/GrpcSpanExporterBuilderWrapper.java
@@ -7,6 +7,7 @@
 
 import io.grpc.ManagedChannel;
 import io.opentelemetry.api.metrics.MeterProvider;
+import io.opentelemetry.common.ComponentLoader;
 import io.opentelemetry.exporter.otlp.trace.OtlpGrpcSpanExporterBuilder;
 import io.opentelemetry.sdk.common.InternalTelemetryVersion;
 import io.opentelemetry.sdk.common.export.ProxyOptions;
@@ -122,6 +123,12 @@ public TelemetryExporterBuilder<SpanData> setServiceClassLoader(ClassLoader serv
     return this;
   }
 
+  @Override
+  public TelemetryExporterBuilder<SpanData> setComponentLoader(ComponentLoader componentLoader) {
+    builder.setComponentLoader(componentLoader);
+    return this;
+  }
+
   @Override
   public TelemetryExporterBuilder<SpanData> setExecutorService(ExecutorService executorService) {
     builder.setExecutorService(executorService);
diff --git a/exporters/otlp/testing-internal/src/main/java/io/opentelemetry/exporter/otlp/testing/internal/HttpLogRecordExporterBuilderWrapper.java b/exporters/otlp/testing-internal/src/main/java/io/opentelemetry/exporter/otlp/testing/internal/HttpLogRecordExporterBuilderWrapper.java
@@ -6,6 +6,7 @@
 package io.opentelemetry.exporter.otlp.testing.internal;
 
 import io.opentelemetry.api.metrics.MeterProvider;
+import io.opentelemetry.common.ComponentLoader;
 import io.opentelemetry.exporter.otlp.http.logs.OtlpHttpLogRecordExporterBuilder;
 import io.opentelemetry.sdk.common.InternalTelemetryVersion;
 import io.opentelemetry.sdk.common.export.ProxyOptions;
@@ -122,6 +123,13 @@ public TelemetryExporterBuilder<LogRecordData> setServiceClassLoader(
     return this;
   }
 
+  @Override
+  public TelemetryExporterBuilder<LogRecordData> setComponentLoader(
+      ComponentLoader componentLoader) {
+    builder.setComponentLoader(componentLoader);
+    return this;
+  }
+
   @Override
   public TelemetryExporterBuilder<LogRecordData> setExecutorService(
       ExecutorService executorService) {
diff --git a/exporters/otlp/testing-internal/src/main/java/io/opentelemetry/exporter/otlp/testing/internal/HttpMetricExporterBuilderWrapper.java b/exporters/otlp/testing-internal/src/main/java/io/opentelemetry/exporter/otlp/testing/internal/HttpMetricExporterBuilderWrapper.java
@@ -6,6 +6,7 @@
 package io.opentelemetry.exporter.otlp.testing.internal;
 
 import io.opentelemetry.api.metrics.MeterProvider;
+import io.opentelemetry.common.ComponentLoader;
 import io.opentelemetry.exporter.otlp.http.metrics.OtlpHttpMetricExporterBuilder;
 import io.opentelemetry.sdk.common.InternalTelemetryVersion;
 import io.opentelemetry.sdk.common.export.ProxyOptions;
@@ -121,6 +122,12 @@ public TelemetryExporterBuilder<MetricData> setServiceClassLoader(
     return this;
   }
 
+  @Override
+  public TelemetryExporterBuilder<MetricData> setComponentLoader(ComponentLoader componentLoader) {
+    builder.setComponentLoader(componentLoader);
+    return this;
+  }
+
   @Override
   public TelemetryExporterBuilder<MetricData> setExecutorService(ExecutorService executorService) {
     builder.setExecutorService(executorService);
diff --git a/exporters/otlp/testing-internal/src/main/java/io/opentelemetry/exporter/otlp/testing/internal/HttpSpanExporterBuilderWrapper.java b/exporters/otlp/testing-internal/src/main/java/io/opentelemetry/exporter/otlp/testing/internal/HttpSpanExporterBuilderWrapper.java
@@ -6,6 +6,7 @@
 package io.opentelemetry.exporter.otlp.testing.internal;
 
 import io.opentelemetry.api.metrics.MeterProvider;
+import io.opentelemetry.common.ComponentLoader;
 import io.opentelemetry.exporter.otlp.http.trace.OtlpHttpSpanExporterBuilder;
 import io.opentelemetry.sdk.common.InternalTelemetryVersion;
 import io.opentelemetry.sdk.common.export.ProxyOptions;
@@ -120,6 +121,12 @@ public TelemetryExporterBuilder<SpanData> setServiceClassLoader(ClassLoader serv
     return this;
   }
 
+  @Override
+  public TelemetryExporterBuilder<SpanData> setComponentLoader(ComponentLoader componentLoader) {
+    builder.setComponentLoader(componentLoader);
+    return this;
+  }
+
   @Override
   public TelemetryExporterBuilder<SpanData> setExecutorService(ExecutorService executorService) {
     builder.setExecutorService(executorService);
diff --git a/exporters/otlp/testing-internal/src/main/java/io/opentelemetry/exporter/otlp/testing/internal/ManagedChannelTelemetryExporterBuilder.java b/exporters/otlp/testing-internal/src/main/java/io/opentelemetry/exporter/otlp/testing/internal/ManagedChannelTelemetryExporterBuilder.java
@@ -13,6 +13,7 @@
 import io.grpc.netty.NettyChannelBuilder;
 import io.netty.handler.ssl.SslContext;
 import io.opentelemetry.api.metrics.MeterProvider;
+import io.opentelemetry.common.ComponentLoader;
 import io.opentelemetry.exporter.internal.TlsConfigHelper;
 import io.opentelemetry.exporter.internal.grpc.ManagedChannelUtil;
 import io.opentelemetry.exporter.otlp.internal.OtlpUserAgent;
@@ -168,6 +169,12 @@ public TelemetryExporterBuilder<T> setServiceClassLoader(ClassLoader serviceClas
     return this;
   }
 
+  @Override
+  public TelemetryExporterBuilder<T> setComponentLoader(ComponentLoader componentLoader) {
+    delegate.setComponentLoader(componentLoader);
+    return this;
+  }
+
   @Override
   public TelemetryExporterBuilder<T> setExecutorService(ExecutorService executorService) {
     delegate.setExecutorService(executorService);
diff --git a/exporters/otlp/testing-internal/src/main/java/io/opentelemetry/exporter/otlp/testing/internal/TelemetryExporterBuilder.java b/exporters/otlp/testing-internal/src/main/java/io/opentelemetry/exporter/otlp/testing/internal/TelemetryExporterBuilder.java
@@ -6,6 +6,7 @@
 package io.opentelemetry.exporter.otlp.testing.internal;
 
 import io.opentelemetry.api.metrics.MeterProvider;
+import io.opentelemetry.common.ComponentLoader;
 import io.opentelemetry.exporter.otlp.logs.OtlpGrpcLogRecordExporterBuilder;
 import io.opentelemetry.exporter.otlp.metrics.OtlpGrpcMetricExporterBuilder;
 import io.opentelemetry.exporter.otlp.profiles.OtlpGrpcProfilesExporterBuilder;
@@ -74,6 +75,8 @@ static TelemetryExporterBuilder<ProfileData> wrap(OtlpGrpcProfilesExporterBuilde
 
   TelemetryExporterBuilder<T> setServiceClassLoader(ClassLoader serviceClassLoader);
 
+  TelemetryExporterBuilder<T> setComponentLoader(ComponentLoader componentLoader);
+
   TelemetryExporterBuilder<T> setExecutorService(ExecutorService executorService);
 
   TelemetryExporterBuilder<T> setMeterProvider(Supplier<MeterProvider> meterProviderSupplier);

Original file line number	Diff line number	Diff line change
`@@ -205,6 +205,7 @@ public GrpcExporterBuilder<T> copy() {`
`205`	`205`	`copy.meterProviderSupplier = meterProviderSupplier;`
`206`	`206`	`copy.internalTelemetryVersion = internalTelemetryVersion;`
`207`	`207`	`copy.grpcChannel = grpcChannel;`
	`208`	`+ copy.componentLoader = componentLoader;`
`208`	`209`	`return copy;`
`209`	`210`	`}`
`210`	`211`
Original file line number	Diff line number	Diff line change
`@@ -204,6 +204,7 @@ public HttpExporterBuilder<T> copy() {`
`204`	`204`	`copy.meterProviderSupplier = meterProviderSupplier;`
`205`	`205`	`copy.internalTelemetryVersion = internalTelemetryVersion;`
`206`	`206`	`copy.proxyOptions = proxyOptions;`
	`207`	`+ copy.componentLoader = componentLoader;`
`207`	`208`	`return copy;`
`208`	`209`	`}`
`209`	`210`