Disable jaxrs annotation instrumentation when controller telemetry is not enabled (#10203)

Author: laurit

instrumentation/jaxrs/jaxrs-1.0/javaagent/build.gradle.kts

@@ -25,6 +25,7 @@ dependencies {
   testImplementation("io.dropwizard:dropwizard-testing:0.7.1")
   testImplementation("javax.xml.bind:jaxb-api:2.2.3")
 }
+
 tasks.withType<Test>().configureEach {
   jvmArgs("-Dotel.instrumentation.common.experimental.controller-telemetry.enabled=true")
 }

instrumentation/jaxrs/jaxrs-1.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jaxrs/v1_0/JaxrsInstrumentationModule.java

@@ -10,8 +10,10 @@
 import static net.bytebuddy.matcher.ElementMatchers.not;
 
 import com.google.auto.service.AutoService;
+import io.opentelemetry.javaagent.bootstrap.internal.ExperimentalConfig;
 import io.opentelemetry.javaagent.extension.instrumentation.InstrumentationModule;
 import io.opentelemetry.javaagent.extension.instrumentation.TypeInstrumentation;
+import io.opentelemetry.sdk.autoconfigure.spi.ConfigProperties;
 import java.util.List;
 import net.bytebuddy.matcher.ElementMatcher;
 
@@ -31,4 +33,12 @@ public ElementMatcher.Junction<ClassLoader> classLoaderMatcher() {
   public List<TypeInstrumentation> typeInstrumentations() {
     return singletonList(new JaxrsAnnotationsInstrumentation());
   }
+
+  @Override
+  public boolean defaultEnabled(ConfigProperties config) {
+    // This instrumentation produces controller telemetry and sets http route. Http route is set by
+    // this instrumentation only when it was not already set by a jax-rs framework instrumentation.
+    // This instrumentation uses complex type matcher, disabling it can improve startup performance.
+    return ExperimentalConfig.get().controllerTelemetryEnabled();
+  }
 }

instrumentation/jaxrs/jaxrs-2.0/jaxrs-2.0-annotations/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jaxrs/v2_0/JaxrsAnnotationsInstrumentationModule.java

@@ -9,8 +9,10 @@
 import static java.util.Arrays.asList;
 
 import com.google.auto.service.AutoService;
+import io.opentelemetry.javaagent.bootstrap.internal.ExperimentalConfig;
 import io.opentelemetry.javaagent.extension.instrumentation.InstrumentationModule;
 import io.opentelemetry.javaagent.extension.instrumentation.TypeInstrumentation;
+import io.opentelemetry.sdk.autoconfigure.spi.ConfigProperties;
 import java.util.List;
 import net.bytebuddy.matcher.ElementMatcher;
 
@@ -34,4 +36,12 @@ public List<TypeInstrumentation> typeInstrumentations() {
         new JaxrsAnnotationsInstrumentation(),
         new JaxrsAsyncResponseInstrumentation());
   }
+
+  @Override
+  public boolean defaultEnabled(ConfigProperties config) {
+    // This instrumentation produces controller telemetry and sets http route. Http route is set by
+    // this instrumentation only when it was not already set by a jax-rs framework instrumentation.
+    // This instrumentation uses complex type matcher, disabling it can improve startup performance.
+    return ExperimentalConfig.get().controllerTelemetryEnabled();
+  }
 }

instrumentation/jaxrs/jaxrs-3.0/jaxrs-3.0-annotations/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jaxrs/v3_0/JaxrsAnnotationsInstrumentationModule.java

@@ -9,8 +9,10 @@
 import static java.util.Arrays.asList;
 
 import com.google.auto.service.AutoService;
+import io.opentelemetry.javaagent.bootstrap.internal.ExperimentalConfig;
 import io.opentelemetry.javaagent.extension.instrumentation.InstrumentationModule;
 import io.opentelemetry.javaagent.extension.instrumentation.TypeInstrumentation;
+import io.opentelemetry.sdk.autoconfigure.spi.ConfigProperties;
 import java.util.List;
 import net.bytebuddy.matcher.ElementMatcher;
 
@@ -34,4 +36,12 @@ public List<TypeInstrumentation> typeInstrumentations() {
         new JaxrsAnnotationsInstrumentation(),
         new JaxrsAsyncResponseInstrumentation());
   }
+
+  @Override
+  public boolean defaultEnabled(ConfigProperties config) {
+    // This instrumentation produces controller telemetry and sets http route. Http route is set by
+    // this instrumentation only when it was not already set by a jax-rs framework instrumentation.
+    // This instrumentation uses complex type matcher, disabling it can improve startup performance.
+    return ExperimentalConfig.get().controllerTelemetryEnabled();
+  }
 }