Skip to content

build: Java 21 toolchain and ubuntu-24.04 CI - #374

Merged
Kunal-8789 merged 2 commits into
mainfrom
fm/java21-toolchain
Oct 7, 2026
Merged

Kunal-8789 merged 2 commits into
mainfrom
fm/java21-toolchain

Conversation

@Kunal-8789

@Kunal-8789 Kunal-8789 commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Why

Lock refresh PRs (e.g. #373) have been failing since May because CI runs on ubuntu-22.04 with no JDK setup (default JDK 11). Jetty 12 requires Java 17+ at runtime. The docker plugin would still emit hypertrace/java:11 without the convention plugin, which would crash on startup with current BOM lines. Stale locks on main also left the shipped image with Trivy HIGH/CRITICAL findings (jackson, netty, jetty 11, micrometer, postgresql).

What

  • Apply org.hypertrace.java-convention 0.4.0 (JDK 21 toolchain, --release 11) and remove manual Java 11 compatibility settings.
  • Move all five Gradle workflows from ubuntu-22.04 to ubuntu-24.04.
  • Shipped image base becomes hypertrace/java:21 via the convention plugin (verified locally with generateJavaApplicationDockerfile).
  • Refresh all module gradle.lockfile entries (same resolution as lock PR Update gradle locks (Automated) #373): jackson 2.21.7, netty 4.1.138, jetty 12.x, micrometer 1.15.12, postgresql 42.7.13.

Verified locally on JDK 21: ./gradlew build and :config-service:integrationTest (Docker/colima) after lock refresh.

Jira: APPSEC-1213

Made with Cursor

Use org.hypertrace.java-convention 0.4.0 (JDK 21 toolchain, release 11)
so Jetty 12+ dependencies and hypertrace/java:21 images work in CI.
Move workflows from ubuntu-22.04 (default JDK 11) to ubuntu-24.04.

APPSEC-1213
@Kunal-8789
Kunal-8789 requested a review from a team as a code owner October 7, 2026 09:11
@github-actions

github-actions Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Test Results

169 tests  ±0   169 ✅ ±0   57s ⏱️ -5s
 31 suites ±0     0 💤 ±0 
 31 files   ±0     0 ❌ ±0 

Results for commit b0efd44. ± Comparison against base commit 206c39a.

♻️ This comment has been updated with latest results.

Align dependency locks with hypertrace-bom resolution (jackson 2.21.7,
netty 4.1.138, jetty 12.x, micrometer 1.15.12, postgresql 42.7.13)
so Trivy image scan passes on the shipped config-service image.

APPSEC-1213
@codecov

codecov Bot commented Oct 7, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.09%. Comparing base (943e5ae) to head (b0efd44).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #374      +/-   ##
============================================
+ Coverage     75.64%   76.09%   +0.44%     
- Complexity      458      625     +167     
============================================
  Files            65       65              
  Lines          3039     3070      +31     
  Branches        149      158       +9     
============================================
+ Hits           2299     2336      +37     
+ Misses          647      644       -3     
+ Partials         93       90       -3     
Flag Coverage Δ
integration 76.09% <ø> (+0.44%) ⬆️
unit 74.09% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Kunal-8789
Kunal-8789 merged commit e409e26 into main Oct 7, 2026
13 of 14 checks passed
@Kunal-8789
Kunal-8789 deleted the fm/java21-toolchain branch October 7, 2026 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants