Skip to content

Bump Gradle daemon JDK to 25#11570

Open
bric3 wants to merge 3 commits into
masterfrom
bdu/bump-daemon-jdk-25
Open

Bump Gradle daemon JDK to 25#11570
bric3 wants to merge 3 commits into
masterfrom
bdu/bump-daemon-jdk-25

Conversation

@bric3
Copy link
Copy Markdown
Contributor

@bric3 bric3 commented Jun 4, 2026

What Does This Do

Bumps the Gradle daemon JDK from 21 to 25 via ./gradlew updateDaemonJvm --jvm-version=25.

Motivation

Keep the build daemon on the latest JDK release.

Additional Notes

  • Kotlin Gradle plugin bumped from 2.0.21 to 2.1.20, Kotlin 2.1.x is the first to support running on Java 25
  • javadoc task now explicitly uses the JDK 8 toolchain, the JDK 25 javadoc treats HTTP redirects as hard errors, and it happens that the Oracle Java 8 API link appears to be broken; running with JDK 8's own tool avoids this.

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
@bric3 bric3 added comp: core Tracer core tag: no release notes Changes to exclude from release notes type: refactoring labels Jun 4, 2026
@datadog-datadog-prod-us1-2

This comment has been minimized.

@dd-octo-sts
Copy link
Copy Markdown
Contributor

dd-octo-sts Bot commented Jun 4, 2026

🟢 Java Benchmark SLOs — All performance SLOs passed

Suite Status
Startup 🟢 pass

SLO thresholds are defined here based on automatically generated metrics. A warning is raised when results are within 5% of the threshold.

PR vs. master results
Scenario Candidate master Δ (95% CI of mean)
startup:insecure-bank:iast:Agent 13.97 s 13.99 s [-1.1%; +0.8%] (no difference)
startup:insecure-bank:tracing:Agent 12.99 s 13.03 s [-1.3%; +0.7%] (no difference)
startup:petclinic:appsec:Agent 16.47 s 16.36 s [-0.6%; +2.0%] (no difference)
startup:petclinic:iast:Agent 16.60 s 16.66 s [-1.3%; +0.6%] (no difference)
startup:petclinic:profiling:Agent 16.52 s 16.63 s [-2.1%; +0.8%] (no difference)
startup:petclinic:tracing:Agent 15.98 s 16.06 s [-1.5%; +0.5%] (no difference)

Commit: f4d61fdb · CI Pipeline · Benchmarking Platform UI


Load and DaCapo benchmarks can be triggered manually in the GitLab pipeline. Results will appear in the Benchmarking Platform UI after completion.

bric3 and others added 2 commits June 4, 2026 21:44
Kotlin 2.0.21 fails to parse JDK 25 version strings (IllegalArgumentException
in JavaVersion.parse). Kotlin 2.1.20 resolves this.

JDK 25 javadoc rejects the old Oracle Java 8 docs URL which now redirects;
replace with the stable canonical URL.

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
JDK 25 javadoc treats HTTP redirects as hard errors; Oracle's Java 8
API URL now redirects. Running javadoc with the JDK 8 tool is
semantically correct (source=8) and avoids the redirect restriction.

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
@bric3 bric3 added comp: tooling Build & Tooling tag: ai generated Largely based on code generated by an AI or LLM and removed comp: core Tracer core type: refactoring labels Jun 4, 2026
@bric3 bric3 marked this pull request as ready for review June 4, 2026 21:31
@bric3 bric3 requested a review from a team as a code owner June 4, 2026 21:31
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f4d61fdbe6

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread gradle/libs.versions.toml
## Kotlin
kotlin = "1.6.21"
kotlin-plugin = "2.0.21"
kotlin-plugin = "2.1.20"
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Refresh Kotlin lockfiles with the plugin bump

When this bumps the Kotlin Gradle plugin to 2.1.20 for Java 25 support, the existing Kotlin module lockfiles are left pinning the Kotlin build-tool configurations to 2.0.21 (checked dd-java-agent/agent-ci-visibility, dd-smoke-tests/iast-propagation, dd-java-agent/instrumentation/spring/spring-messaging-4.0, dd-java-agent/instrumentation/kotlin-coroutines-1.3, and dd-java-agent/instrumentation/junit/junit-4/junit-4.10). Because this repo's dependency locking applies whenever configurations such as kotlinCompilerClasspath are resolved, those modules can still resolve the old Java-25-incompatible compiler/daemon artifacts despite the catalog saying 2.1.20; please regenerate and commit the affected gradle.lockfile files with --write-locks.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp: tooling Build & Tooling tag: ai generated Largely based on code generated by an AI or LLM tag: no release notes Changes to exclude from release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant