Skip to content

Update bundled JDK to 21.0.12+8 - #22572

Merged
andrross merged 1 commit into
opensearch-project:2.19from
reta:issue-22532.2.19
Jul 30, 2026
Merged

Update bundled JDK to 21.0.12+8#22572
andrross merged 1 commit into
opensearch-project:2.19from
reta:issue-22532.2.19

Conversation

@reta

@reta reta commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Description

Update bundled JDK to 21.0.12+8 (tracking release binaries https://github.com/adoptium/temurin21-binaries/releases/tag/jdk-21.0.12%2B8)

Related Issues

Part of #22532

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@reta reta added the Build Build Tasks/Gradle Plugin, groovy scripts, build tools, Javadoc enforcement. label Jul 27, 2026
@github-actions

Copy link
Copy Markdown
Contributor

PR Code Analyzer ❗

AI-powered 'Code-Diff-Analyzer' found issues on commit 82640b7.

PathLineSeverityDescription
buildSrc/src/main/java/org/opensearch/gradle/test/DistroTestPlugin.java80highBundled JDK version changed from 21.0.11+10 to 21.0.12+8 (vendor: adoptium) for both SYSTEM_JDK_VERSION and GRADLE_JDK_VERSION. Per mandatory rule, all dependency version changes must be flagged. Maintainers should verify the new JDK artifact hash/signature matches the official Adoptium release for 21.0.12+8.
gradle/libs.versions.toml6highbundled_jdk version changed from 21.0.11+10 to 21.0.12+8 (vendor: adoptium). Per mandatory rule, all dependency version changes must be flagged. This controls which JDK artifact is fetched at build time; maintainers should confirm the artifact origin and integrity (checksums/signatures) against the official Adoptium Temurin release.

The table above displays the top 10 most important findings.

Total: 2 | Critical: 0 | High: 2 | Medium: 0 | Low: 0


Pull Requests Author(s): Please update your Pull Request according to the report above.

Repository Maintainer(s): You can bypass diff analyzer by adding label skip-diff-analyzer after reviewing the changes carefully, then re-run failed actions. To re-enable the analyzer, remove the label, then re-run all actions.


⚠️ Note: The Code-Diff-Analyzer helps protect against potentially harmful code patterns. Please ensure you have thoroughly reviewed the changes beforehand.

Thanks.

Signed-off-by: Andriy Redko <drreta@gmail.com>
@reta
reta force-pushed the issue-22532.2.19 branch from 82640b7 to 7321be1 Compare July 27, 2026 14:56
@reta reta added the skip-diff-analyzer Maintainer to skip code-diff-analyzer check, after reviewing issues in AI analysis. label Jul 27, 2026
@github-actions

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🧪 No relevant tests
🔒 No security concerns identified
✅ No TODO sections
🔀 No multiple PR themes
⚡ No major issues detected

@github-actions

Copy link
Copy Markdown
Contributor

❌ Gradle check result for 7321be1: TIMEOUT

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions

Copy link
Copy Markdown
Contributor

❌ Gradle check result for 7321be1: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@reta
reta marked this pull request as ready for review July 30, 2026 14:54
@reta
reta requested a review from a team as a code owner July 30, 2026 14:54
@github-actions

Copy link
Copy Markdown
Contributor

✅ Gradle check result for 7321be1: SUCCESS

@codecov

codecov Bot commented Jul 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.03%. Comparing base (aff3489) to head (7321be1).
⚠️ Report is 21 commits behind head on 2.19.

Additional details and impacted files
@@             Coverage Diff              @@
##               2.19   #22572      +/-   ##
============================================
+ Coverage     71.92%   72.03%   +0.10%     
+ Complexity    66009    64520    -1489     
============================================
  Files          5342     5120     -222     
  Lines        307392   300237    -7155     
  Branches      44862    44102     -760     
============================================
- Hits         221105   216275    -4830     
+ Misses        67823    65865    -1958     
+ Partials      18464    18097     -367     

☔ 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.

@reta

reta commented Jul 30, 2026

Copy link
Copy Markdown
Contributor Author

@cwperks @andrross would appreciate a look folks, when time permits, thanks!

@andrross
andrross merged commit 7c68a18 into opensearch-project:2.19 Jul 30, 2026
83 of 150 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Build Build Tasks/Gradle Plugin, groovy scripts, build tools, Javadoc enforcement. skip-diff-analyzer Maintainer to skip code-diff-analyzer check, after reviewing issues in AI analysis.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants