Review fixes for runtime-telemetry-java17:javaagent (#17663)

otelbot[bot] · web-flow · commit d63c7ad78a88 · 2026-04-07T10:51:04.000-07:00
Co-authored-by: otelbot &lt;197425009+otelbot@users.noreply.github.com&gt;
diff --git a/instrumentation/runtime-telemetry/runtime-telemetry-java17/javaagent/build.gradle.kts b/instrumentation/runtime-telemetry/runtime-telemetry-java17/javaagent/build.gradle.kts
@@ -10,6 +10,7 @@ otelJava {
 }
 
 dependencies {
+  testInstrumentation(project(":instrumentation:runtime-telemetry:runtime-telemetry-java8:javaagent"))
   testInstrumentation(project(":instrumentation:runtime-telemetry:javaagent"))
 }
 
diff --git a/instrumentation/runtime-telemetry/runtime-telemetry-java17/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/runtimemetrics/java17/JfrRuntimeMetricsTest.java b/instrumentation/runtime-telemetry/runtime-telemetry-java17/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/runtimemetrics/java17/JfrRuntimeMetricsTest.java
@@ -22,7 +22,7 @@ class JfrRuntimeMetricsTest {
   static void setUp() {
     try {
       Class.forName("jdk.jfr.FlightRecorder");
-    } catch (ClassNotFoundException exception) {
+    } catch (ClassNotFoundException e) {
       Assumptions.abort("JFR not present");
     }
     Assumptions.assumeTrue(FlightRecorder.isAvailable(), "JFR not available");

Original file line number	Diff line number	Diff line change
`@@ -10,6 +10,7 @@ otelJava {`
`10`	`10`	`}`
`11`	`11`
`12`	`12`	`dependencies {`
	`13`	`+ testInstrumentation(project(":instrumentation:runtime-telemetry:runtime-telemetry-java8:javaagent"))`
`13`	`14`	`testInstrumentation(project(":instrumentation:runtime-telemetry:javaagent"))`
`14`	`15`	`}`
`15`	`16`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ class JfrRuntimeMetricsTest {`
`22`	`22`	`static void setUp() {`
`23`	`23`	`try {`
`24`	`24`	`Class.forName("jdk.jfr.FlightRecorder");`
`25`		`- } catch (ClassNotFoundException exception) {`
	`25`	`+ } catch (ClassNotFoundException e) {`
`26`	`26`	`Assumptions.abort("JFR not present");`
`27`	`27`	`}`
`28`	`28`	`Assumptions.assumeTrue(FlightRecorder.isAvailable(), "JFR not available");`