From 1c57667bab9c553df1aa318c5e012610f9e3cd65 Mon Sep 17 00:00:00 2001 From: Ryan Ernst Date: Mon, 11 Aug 2025 07:19:33 -0700 Subject: [PATCH 1/2] Use java8 variant of apm-agent This commit changes the apm-agent dependency to the java8 variant. The current dependency has a minimum java7 version, which restricts the dependency upgrades it can do. --- .../java/org/elasticsearch/server/cli/APMJvmOptions.java | 2 +- .../org/elasticsearch/server/cli/APMJvmOptionsTests.java | 2 +- gradle/verification-metadata.xml | 6 +++--- modules/apm/build.gradle | 2 +- ...gent-LICENSE.txt => elastic-apm-agent-java8-LICENSE.txt} | 0 ...-agent-NOTICE.txt => elastic-apm-agent-java8-NOTICE.txt} | 0 server/src/main/java/org/elasticsearch/node/Node.java | 2 +- 7 files changed, 7 insertions(+), 7 deletions(-) rename modules/apm/licenses/{elastic-apm-agent-LICENSE.txt => elastic-apm-agent-java8-LICENSE.txt} (100%) rename modules/apm/licenses/{elastic-apm-agent-NOTICE.txt => elastic-apm-agent-java8-NOTICE.txt} (100%) diff --git a/distribution/tools/server-cli/src/main/java/org/elasticsearch/server/cli/APMJvmOptions.java b/distribution/tools/server-cli/src/main/java/org/elasticsearch/server/cli/APMJvmOptions.java index 1e57d9fab7cfd..9cf9e38aeed20 100644 --- a/distribution/tools/server-cli/src/main/java/org/elasticsearch/server/cli/APMJvmOptions.java +++ b/distribution/tools/server-cli/src/main/java/org/elasticsearch/server/cli/APMJvmOptions.java @@ -329,7 +329,7 @@ static Path findAgentJar(String installDir) throws IOException, UserException { try (var apmStream = Files.list(apmModule)) { final List paths = apmStream.filter( - path -> path.getFileName().toString().matches("elastic-apm-agent-\\d+\\.\\d+\\.\\d+\\.jar") + path -> path.getFileName().toString().matches("elastic-apm-agent-java8-\\d+\\.\\d+\\.\\d+\\.jar") ).toList(); if (paths.size() > 1) { diff --git a/distribution/tools/server-cli/src/test/java/org/elasticsearch/server/cli/APMJvmOptionsTests.java b/distribution/tools/server-cli/src/test/java/org/elasticsearch/server/cli/APMJvmOptionsTests.java index 1b2ea41cee911..fdb27f79703c0 100644 --- a/distribution/tools/server-cli/src/test/java/org/elasticsearch/server/cli/APMJvmOptionsTests.java +++ b/distribution/tools/server-cli/src/test/java/org/elasticsearch/server/cli/APMJvmOptionsTests.java @@ -141,7 +141,7 @@ private Path makeFakeAgentJar() throws IOException { Path tempFile = createTempFile(); Path apmPathDir = tempFile.getParent().resolve("modules").resolve("apm"); Files.createDirectories(apmPathDir); - Path apmAgentFile = apmPathDir.resolve("elastic-apm-agent-0.0.0.jar"); + Path apmAgentFile = apmPathDir.resolve("elastic-apm-agent-java8-0.0.0.jar"); Files.move(tempFile, apmAgentFile); return tempFile.getParent(); diff --git a/gradle/verification-metadata.xml b/gradle/verification-metadata.xml index 21d8fc8231ee5..28a7d6981a4cb 100644 --- a/gradle/verification-metadata.xml +++ b/gradle/verification-metadata.xml @@ -76,9 +76,9 @@ - - - + + + diff --git a/modules/apm/build.gradle b/modules/apm/build.gradle index 37d42e4b3fb0c..83d988e966899 100644 --- a/modules/apm/build.gradle +++ b/modules/apm/build.gradle @@ -20,7 +20,7 @@ dependencies { implementation "io.opentelemetry:opentelemetry-api:${otelVersion}" implementation "io.opentelemetry:opentelemetry-context:${otelVersion}" implementation "io.opentelemetry:opentelemetry-semconv:${otelSemconvVersion}" - runtimeOnly "co.elastic.apm:elastic-apm-agent:1.55.0" + runtimeOnly "co.elastic.apm:elastic-apm-agent-java8:1.55.0" javaRestTestImplementation project(':modules:apm') javaRestTestImplementation project(':test:framework') diff --git a/modules/apm/licenses/elastic-apm-agent-LICENSE.txt b/modules/apm/licenses/elastic-apm-agent-java8-LICENSE.txt similarity index 100% rename from modules/apm/licenses/elastic-apm-agent-LICENSE.txt rename to modules/apm/licenses/elastic-apm-agent-java8-LICENSE.txt diff --git a/modules/apm/licenses/elastic-apm-agent-NOTICE.txt b/modules/apm/licenses/elastic-apm-agent-java8-NOTICE.txt similarity index 100% rename from modules/apm/licenses/elastic-apm-agent-NOTICE.txt rename to modules/apm/licenses/elastic-apm-agent-java8-NOTICE.txt diff --git a/server/src/main/java/org/elasticsearch/node/Node.java b/server/src/main/java/org/elasticsearch/node/Node.java index 30a32b07f15f0..6e0cade462e88 100644 --- a/server/src/main/java/org/elasticsearch/node/Node.java +++ b/server/src/main/java/org/elasticsearch/node/Node.java @@ -219,7 +219,7 @@ public static void deleteTemporaryApmConfig(JvmInfo jvmInfo, BiConsumer