OpenTelemetry metrics exporter instrumentation should be enabled even…

… with instrument=false
elastic · Sep 20, 2023 · e8bba81 · e8bba81
1 parent e968cf3
commit e8bba81
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/.../main/java/co/elastic/apm/agent/otelmetricsdk/SdkMeterProviderBuilderInstrumentation.java b/.../main/java/co/elastic/apm/agent/otelmetricsdk/SdkMeterProviderBuilderInstrumentation.java
@@ -19,13 +19,13 @@
 package co.elastic.apm.agent.otelmetricsdk;
 
 import co.elastic.apm.agent.impl.ElasticApmTracer;
-import co.elastic.apm.agent.tracer.GlobalTracer;
 import co.elastic.apm.agent.sdk.ElasticApmInstrumentation;
+import co.elastic.apm.agent.sdk.internal.util.LoggerUtils;
 import co.elastic.apm.agent.sdk.logging.Logger;
 import co.elastic.apm.agent.sdk.logging.LoggerFactory;
 import co.elastic.apm.agent.sdk.weakconcurrent.WeakConcurrent;
 import co.elastic.apm.agent.sdk.weakconcurrent.WeakSet;
-import co.elastic.apm.agent.sdk.internal.util.LoggerUtils;
+import co.elastic.apm.agent.tracer.GlobalTracer;
 import io.opentelemetry.sdk.metrics.SdkMeterProviderBuilder;
 import io.opentelemetry.sdk.metrics.export.MetricExporter;
 import net.bytebuddy.asm.Advice;
@@ -55,6 +55,11 @@ public Collection<String> getInstrumentationGroupNames() {
         return Arrays.asList("opentelemetry", "opentelemetry-metrics", "experimental");
     }
 
+    @Override
+    public final boolean includeWhenInstrumentationIsDisabled() {
+        return true;
+    }
+
     @Override
     public String getAdviceClassName() {
         return getClass().getName() + "$SdkMeterProviderBuilderAdvice";