make opentelemetry-* indy-ready

instrumentation/opentelemetry-api/opentelemetry-api-1.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/opentelemetryapi/ContextInstrumentation.java

@@ -15,6 +15,7 @@
 import io.opentelemetry.javaagent.extension.instrumentation.TypeTransformer;
 import io.opentelemetry.javaagent.instrumentation.opentelemetryapi.context.AgentContextStorage;
 import net.bytebuddy.asm.Advice;
+import net.bytebuddy.asm.Advice.AssignReturned;
 import net.bytebuddy.description.type.TypeDescription;
 import net.bytebuddy.matcher.ElementMatcher;
 
@@ -41,9 +42,10 @@ public void transform(TypeTransformer transformer) {
   @SuppressWarnings("unused")
   public static class WrapRootAdvice {
 
+    @AssignReturned.ToReturned
     @Advice.OnMethodExit(onThrowable = Throwable.class, suppress = Throwable.class)
-    public static void methodExit(@Advice.Return(readOnly = false) Context root) {
-      root = AgentContextStorage.wrapRootContext(root);
+    public static Context methodExit(@Advice.Return Context root) {
+      return AgentContextStorage.wrapRootContext(root);
     }
   }
 }

instrumentation/opentelemetry-api/opentelemetry-api-1.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/opentelemetryapi/ContextStorageWrappersInstrumentation.java

@@ -15,6 +15,7 @@
 import java.util.List;
 import java.util.function.Function;
 import net.bytebuddy.asm.Advice;
+import net.bytebuddy.asm.Advice.AssignReturned;
 import net.bytebuddy.description.type.TypeDescription;
 import net.bytebuddy.matcher.ElementMatcher;
 
@@ -41,13 +42,16 @@ public void transform(TypeTransformer transformer) {
   @SuppressWarnings("unused")
   public static class AddWrapperAdvice {
 
+    @AssignReturned.ToReturned
     @Advice.OnMethodExit(onThrowable = Throwable.class, suppress = Throwable.class)
-    public static void methodExit(
-        @Advice.Return(readOnly = false)
-            List<Function<? super ContextStorage, ? extends ContextStorage>> wrappers) {
+    public static List<Function<? super ContextStorage, ? extends ContextStorage>> methodExit(
+        @Advice.Return
+            List<Function<? super ContextStorage, ? extends ContextStorage>> originalWrappers) {
+      List<Function<? super ContextStorage, ? extends ContextStorage>> wrappers = originalWrappers;
       wrappers = new ArrayList<>(wrappers);
       // AgentContextStorage wrapper doesn't delegate, so needs to be the innermost wrapper
       wrappers.add(0, AgentContextStorage.wrap());
+      return wrappers;
     }
   }
 }

instrumentation/opentelemetry-api/opentelemetry-api-1.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/opentelemetryapi/InstrumentationUtilInstrumentation.java

@@ -15,6 +15,7 @@
 import io.opentelemetry.javaagent.extension.instrumentation.TypeTransformer;
 import io.opentelemetry.javaagent.instrumentation.opentelemetryapi.context.AgentContextStorage;
 import net.bytebuddy.asm.Advice;
+import net.bytebuddy.asm.Advice.AssignReturned;
 import net.bytebuddy.description.type.TypeDescription;
 import net.bytebuddy.matcher.ElementMatcher;
 
@@ -44,12 +45,11 @@ public static boolean methodEnter() {
       return true;
     }
 
+    @AssignReturned.ToReturned
     @Advice.OnMethodExit(suppress = Throwable.class)
-    public static void methodExit(
-        @Advice.Argument(0) Context context, @Advice.Return(readOnly = false) boolean result) {
-      result =
-          InstrumentationUtil.shouldSuppressInstrumentation(
-              AgentContextStorage.getAgentContext(context));
+    public static boolean methodExit(@Advice.Argument(0) Context context) {
+      return InstrumentationUtil.shouldSuppressInstrumentation(
+          AgentContextStorage.getAgentContext(context));
     }
   }
 

instrumentation/opentelemetry-api/opentelemetry-api-1.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/opentelemetryapi/OpenTelemetryApiInstrumentationModule.java

@@ -43,4 +43,9 @@ public List<String> agentPackagesToHide() {
     // when they haven't been injected
     return Collections.singletonList("io.opentelemetry.javaagent.instrumentation.opentelemetryapi");
   }
+
+  @Override
+  public boolean isIndyReady() {
+    return true;
+  }
 }

instrumentation/opentelemetry-api/opentelemetry-api-1.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/opentelemetryapi/OpenTelemetryInstrumentation.java

@@ -17,6 +17,7 @@
 import io.opentelemetry.javaagent.extension.instrumentation.TypeTransformer;
 import java.util.logging.Logger;
 import net.bytebuddy.asm.Advice;
+import net.bytebuddy.asm.Advice.AssignReturned;
 import net.bytebuddy.description.type.TypeDescription;
 import net.bytebuddy.matcher.ElementMatcher;
 
@@ -58,11 +59,10 @@ public static Object onEnter() {
       return null;
     }
 
+    @AssignReturned.ToReturned
     @Advice.OnMethodExit(onThrowable = Throwable.class, suppress = Throwable.class)
-    public static void methodExit(
-        @Advice.Return(readOnly = false)
-            application.io.opentelemetry.api.OpenTelemetry openTelemetry) {
-      openTelemetry = ApplicationOpenTelemetry.INSTANCE;
+    public static application.io.opentelemetry.api.OpenTelemetry methodExit() {
+      return ApplicationOpenTelemetry.INSTANCE;
     }
   }
 

instrumentation/opentelemetry-api/opentelemetry-api-1.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/opentelemetryapi/SpanInstrumentation.java

@@ -15,6 +15,7 @@
 import io.opentelemetry.javaagent.extension.instrumentation.TypeTransformer;
 import io.opentelemetry.javaagent.instrumentation.opentelemetryapi.trace.Bridging;
 import net.bytebuddy.asm.Advice;
+import net.bytebuddy.asm.Advice.AssignReturned;
 import net.bytebuddy.description.type.TypeDescription;
 import net.bytebuddy.matcher.ElementMatcher;
 
@@ -40,13 +41,11 @@ public static boolean methodEnter() {
       return false;
     }
 
+    @AssignReturned.ToReturned
     @Advice.OnMethodExit
-    public static void methodExit(
-        @Advice.Argument(0) SpanContext applicationSpanContext,
-        @Advice.Return(readOnly = false) Span applicationSpan) {
-      applicationSpan =
-          Bridging.toApplication(
-              io.opentelemetry.api.trace.Span.wrap(Bridging.toAgent(applicationSpanContext)));
+    public static Span methodExit(@Advice.Argument(0) SpanContext applicationSpanContext) {
+      return Bridging.toApplication(
+          io.opentelemetry.api.trace.Span.wrap(Bridging.toAgent(applicationSpanContext)));
     }
   }
 }

instrumentation/opentelemetry-api/opentelemetry-api-1.0/testing/src/main/java/io/opentelemetry/javaagent/instrumentation/opentelemetryapi/TestInstrumentation.java

@@ -13,6 +13,7 @@
 import io.opentelemetry.javaagent.extension.instrumentation.TypeTransformer;
 import io.opentelemetry.javaagent.instrumentation.opentelemetryapi.context.AgentContextStorage;
 import net.bytebuddy.asm.Advice;
+import net.bytebuddy.asm.Advice.AssignReturned;
 import net.bytebuddy.description.type.TypeDescription;
 import net.bytebuddy.matcher.ElementMatcher;
 
@@ -31,12 +32,11 @@ public void transform(TypeTransformer transformer) {
   @SuppressWarnings("unused")
   public static class TestAdvice {
 
+    @AssignReturned.ToReturned
     @Advice.OnMethodExit(suppress = Throwable.class)
-    public static void onExit(
-        @Advice.Argument(0) Context context, @Advice.Return(readOnly = false) boolean result) {
-      result =
-          InstrumentationUtil.shouldSuppressInstrumentation(
-              AgentContextStorage.getAgentContext(context));
+    public static boolean onExit(@Advice.Argument(0) Context context) {
+      return InstrumentationUtil.shouldSuppressInstrumentation(
+          AgentContextStorage.getAgentContext(context));
     }
   }
 }

instrumentation/opentelemetry-api/opentelemetry-api-1.0/testing/src/main/java/io/opentelemetry/javaagent/instrumentation/opentelemetryapi/TestInstrumentationModule.java

@@ -29,4 +29,9 @@ public String getModuleGroup() {
   public List<TypeInstrumentation> typeInstrumentations() {
     return singletonList(new TestInstrumentation());
   }
+
+  @Override
+  public boolean isIndyReady() {
+    return true;
+  }
 }

instrumentation/opentelemetry-api/opentelemetry-api-1.10/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/opentelemetryapi/v1_10/OpenTelemetryApiInstrumentationModule.java

@@ -37,4 +37,9 @@ public ElementMatcher.Junction<ClassLoader> classLoaderMatcher() {
   public String getModuleGroup() {
     return "opentelemetry-api-bridge";
   }
+
+  @Override
+  public boolean isIndyReady() {
+    return true;
+  }
 }

instrumentation/opentelemetry-api/opentelemetry-api-1.31/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/opentelemetryapi/v1_31/incubator/OpenTelemetryApiInstrumentationModule.java

@@ -37,4 +37,9 @@ public ElementMatcher.Junction<ClassLoader> classLoaderMatcher() {
   public String getModuleGroup() {
     return "opentelemetry-api-bridge";
   }
+
+  @Override
+  public boolean isIndyReady() {
+    return true;
+  }
 }

instrumentation/opentelemetry-api/opentelemetry-api-1.32/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/opentelemetryapi/v1_32/OpenTelemetryApiInstrumentationModule.java

@@ -43,4 +43,9 @@ public List<TypeInstrumentation> typeInstrumentations() {
   public String getModuleGroup() {
     return "opentelemetry-api-bridge";
   }
+
+  @Override
+  public boolean isIndyReady() {
+    return true;
+  }
 }

instrumentation/opentelemetry-api/opentelemetry-api-1.32/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/opentelemetryapi/v1_32/incubator/OpenTelemetryApiIncubatorInstrumentationModule.java

@@ -39,4 +39,9 @@ public List<TypeInstrumentation> typeInstrumentations() {
   public String getModuleGroup() {
     return "opentelemetry-api-bridge";
   }
+
+  @Override
+  public boolean isIndyReady() {
+    return true;
+  }
 }

instrumentation/opentelemetry-api/opentelemetry-api-1.38/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/opentelemetryapi/v1_38/OpenTelemetryApiInstrumentationModule.java

@@ -43,4 +43,9 @@ public List<TypeInstrumentation> typeInstrumentations() {
   public String getModuleGroup() {
     return "opentelemetry-api-bridge";
   }
+
+  @Override
+  public boolean isIndyReady() {
+    return true;
+  }
 }

instrumentation/opentelemetry-api/opentelemetry-api-1.38/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/opentelemetryapi/v1_38/incubator/OpenTelemetryApiIncubatorInstrumentationModule.java

@@ -40,4 +40,9 @@ public List<TypeInstrumentation> typeInstrumentations() {
   public String getModuleGroup() {
     return "opentelemetry-api-bridge";
   }
+
+  @Override
+  public boolean isIndyReady() {
+    return true;
+  }
 }

instrumentation/opentelemetry-api/opentelemetry-api-1.40/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/opentelemetryapi/v1_40/incubator/OpenTelemetryApiIncubatorInstrumentationModule.java

@@ -40,4 +40,9 @@ public List<TypeInstrumentation> typeInstrumentations() {
   public String getModuleGroup() {
     return "opentelemetry-api-bridge";
   }
+
+  @Override
+  public boolean isIndyReady() {
+    return true;
+  }
 }

instrumentation/opentelemetry-api/opentelemetry-api-1.47/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/opentelemetryapi/v1_47/incubator/OpenTelemetryApiIncubatorInstrumentationModule.java

@@ -43,4 +43,9 @@ public List<TypeInstrumentation> typeInstrumentations() {
   public String getModuleGroup() {
     return "opentelemetry-api-bridge";
   }
+
+  @Override
+  public boolean isIndyReady() {
+    return true;
+  }
 }