Add Apolloconfig Inst

#12787

Author: freshchen

docs/supported-libraries.md

@@ -6,7 +6,7 @@ muzzle {
   pass {
     group.set("com.ctrip.framework.apollo")
     module.set("apollo-client")
-    versions.set("[1.0.0,2.3.0]")
+    versions.set("[1.0.0,)")
     assertInverse.set(true)
   }
 }
@@ -15,15 +15,13 @@ dependencies {
   compileOnly("com.google.auto.value:auto-value-annotations")
   annotationProcessor("com.google.auto.value:auto-value")
 
-  library("com.ctrip.framework.apollo:apollo-client:1.1.0")
+  library("com.ctrip.framework.apollo:apollo-client:1.0.0")
 
+  testImplementation("com.ctrip.framework.apollo:apollo-client:1.1.0")
   testImplementation(project(":testing-common"))
-
-  latestDepTestLibrary("com.ctrip.framework.apollo:apollo-client:1.1.+")
 }
 
 tasks.withType<Test>().configureEach {
-  // required on jdk17
-  jvmArgs("--add-opens=java.base/java.lang=ALL-UNNAMED")
-  jvmArgs("-XX:+IgnoreUnrecognizedVMOptions")
+  jvmArgs("-Dotel.instrumentation.apolloconfig-apolloclient.enabled=true")
 }
+

instrumentation/apolloconfig-apolloclient-1.0/javaagent/build.gradle.kts

@@ -10,6 +10,7 @@
 import com.google.auto.service.AutoService;
 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;
 
 @AutoService(InstrumentationModule.class)
@@ -22,4 +23,9 @@ public ApolloConfigInstrumentationModule() {
   public List<TypeInstrumentation> typeInstrumentations() {
     return singletonList(new ApolloRepositoryChangeInstrumentation());
   }
+
+  @Override
+  public boolean defaultEnabled(ConfigProperties config) {
+    return false;
+  }
 }

instrumentation/apolloconfig-apolloclient-1.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/apolloconfig/apolloclient/v1_0/ApolloConfigInstrumentationModule.java

@@ -63,7 +63,7 @@ public void onEnd(
                   }
                 })
             .addAttributesExtractor(attributesExtractor)
-            .buildInstrumenter(SpanKindExtractor.alwaysClient());
+            .buildInstrumenter(SpanKindExtractor.alwaysInternal());
   }
 
   public static Instrumenter<String, Void> instrumenter() {

instrumentation/apolloconfig-apolloclient-1.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/apolloconfig/apolloclient/v1_0/ApolloConfigSingletons.java

@@ -33,15 +33,11 @@ void fireRepositoryChangeTest() {
     testConfigRepository.addChangeListener(new TestRepositoryChangeListener());
     testConfigRepository.sync();
 
-    checkRepositoryChange(namespace);
-  }
-
-  private static void checkRepositoryChange(String namespace) {
     testing.waitAndAssertTraces(
         trace ->
             trace.hasSpansSatisfyingExactly(
                 span ->
-                    span.hasKind(SpanKind.CLIENT)
+                    span.hasKind(SpanKind.INTERNAL)
                         .hasName("Apollo Config Repository Change")
                         .hasAttributesSatisfyingExactly(
                             singletonList(

instrumentation/apolloconfig-apolloclient-1.0/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/apolloconfig/apolloclient/v1_0/ApolloRepositoryChangeTest.java

@@ -253,7 +253,8 @@ public void configure(IgnoredTypesBuilder builder) {
         // We instrument Runnable there
         .allowClass("com.google.inject.internal.AbstractBindingProcessor$")
         .allowClass("com.google.inject.internal.BytecodeGen$")
-        .allowClass("com.google.inject.internal.cglib.core.internal.$LoadingCache$");
+        .allowClass("com.google.inject.internal.cglib.core.internal.$LoadingCache$")
+        .allowClass("com.google.inject.internal.aop.ChildClassDefiner$ChildLoader");
 
     builder.ignoreClass("com.google.api.").allowClass("com.google.api.client.http.HttpRequest");