Skip to content

Remove Java 8 from the ARM64 GitLab test matrix - #12703

Open
bric3 wants to merge 1 commit into
masterfrom
ci/remove-java8-arm64
Open

bric3 wants to merge 1 commit into
masterfrom
ci/remove-java8-arm64

Conversation

@bric3

@bric3 bric3 commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

What Does This Do

Exclude Java 8 variants from all ARM64 test jobs. Remove the ARM64 latest-dependency Java 8 entry and the resulting unused coverage logic.

Motivation

The newer JVMs contain AArch64 optimizations that might not have been back-ported to the Java 8 distributions we have in had in our test matrix. Moreover depending where the job is scheduled we can land on older hard ware that makes these jobs are way too slow.

It’s noteworthy that AWS explicitly reports that some customers only obtained Graviton's full performance benefit after moving from Java 8 to Java 11. It’s likely possible as well only Corretto 8 (the Amazon Java 8 distribution) got these optimizations, not the other vendors.

In general, here some specific optimizations that may be missing in our chosen Java 8 distributions:

Also, Java 8 lacks broader improvement made in later versions.

Additional Notes

Contributor Checklist

@bric3 bric3 added comp: testing Testing tag: no release notes Changes to exclude from release notes type: refactoring tag: ai generated Largely based on code generated by an AI or LLM labels Sep 30, 2026
@bric3
bric3 marked this pull request as ready for review September 30, 2026 14:39
@bric3
bric3 requested review from a team as code owners September 30, 2026 14:39
@bric3
bric3 requested review from PerfectSlayer and erikayasuda and removed request for a team September 30, 2026 14:39
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-30T14:42:16.784556Z 81788bd Draft marked ready
🔒 Security Review ✅ Completed 2026-09-30T14:44:34.075691Z 81788bd Draft marked ready
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@datadog-official datadog-official Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Bits Code Review: PASS

More details

The ARM64 rules consistently exclude all Java 8 distributions while retaining Java 8 coverage on AMD64; the removed ARM64 coverage branches are consequently unreachable.

Was this helpful? React 👍 or 👎

Open Bits AI session

🤖 Bits Code Review · Commit 81788bd · @DataDog review to ask questions

@datadog-official

This comment has been minimized.

@dd-octo-sts

dd-octo-sts Bot commented Sep 30, 2026

Copy link
Copy Markdown
Contributor

🟢 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 14.92 s 14.75 s [+0.0%; +2.3%] (maybe worse)
startup:insecure-bank:tracing:Agent 13.65 s 13.75 s [-1.5%; -0.0%] (maybe better)
startup:petclinic:appsec:Agent 17.10 s 16.89 s [+0.5%; +2.0%] (maybe worse)
startup:petclinic:iast:Agent 17.03 s 17.06 s [-0.8%; +0.5%] (no difference)
startup:petclinic:profiling:Agent 16.66 s 16.79 s [-2.0%; +0.3%] (no difference)
startup:petclinic:sca:Agent 16.89 s 16.87 s [-0.9%; +1.1%] (no difference)
startup:petclinic:tracing:Agent 15.77 s 16.18 s [-6.6%; +1.7%] (no difference)

Commit: 81788bd0 · 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.

@sarahchen6 sarahchen6 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

This branch has not been deployed

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

Labels

comp: testing Testing tag: ai generated Largely based on code generated by an AI or LLM tag: no release notes Changes to exclude from release notes type: refactoring

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants