Skip to content

Update Gradle to 9.4.1 and Jacoco to 0.8.14 - #21153

Merged
reta merged 1 commit into
opensearch-project:mainfrom
reta:gradle.9.4.1
Apr 7, 2026
Merged

Update Gradle to 9.4.1 and Jacoco to 0.8.14#21153
reta merged 1 commit into
opensearch-project:mainfrom
reta:gradle.9.4.1

Conversation

@reta

@reta reta commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

Description

Update Gradle to 9.4.1 and Jacoco to 0.8.14

Related Issues

Prepare for JDK-26 support

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.

Signed-off-by: Andriy Redko <drreta@gmail.com>
@reta
reta requested a review from a team as a code owner April 7, 2026 15:11
@reta reta added Build Libraries & Interfaces Build Build Tasks/Gradle Plugin, groovy scripts, build tools, Javadoc enforcement. labels Apr 7, 2026
@github-actions

github-actions Bot commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

PR Code Analyzer ❗

AI-powered 'Code-Diff-Analyzer' found issues on commit 8bf11b2.

PathLineSeverityDescription
gradle/wrapper/gradle-wrapper.properties4highGradle wrapper distribution URL changed from gradle-9.4.0-all.zip to gradle-9.4.1-all.zip with an updated SHA256 checksum. Per mandatory policy, all build tool distribution changes must be flagged. Maintainers should verify the new distributionSha256Sum (708d2c6ecc97ca9a11838ef64a6c2301151b8dd10387e22dc1a12c30557cab5b) matches the official Gradle 9.4.1 release artifact at services.gradle.org.
gradle/code-coverage.gradle22highJaCoCo toolVersion bumped from 0.8.13 to 0.8.14. Per mandatory policy, all dependency version changes must be flagged. Maintainers should verify this version exists in the expected Maven Central artifact and that the upgrade is intentional.

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.

@reta reta added the skip-diff-analyzer Maintainer to skip code-diff-analyzer check, after reviewing issues in AI analysis. label Apr 7, 2026
@github-actions

github-actions Bot commented Apr 7, 2026

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
⚡ Recommended focus areas for review

SHA256 Verification

The distributionSha256Sum should be verified against the official Gradle 9.4.1 release checksum to ensure the integrity of the downloaded distribution. Confirm that the provided hash matches the official Gradle 9.4.1 release.

distributionSha256Sum=708d2c6ecc97ca9a11838ef64a6c2301151b8dd10387e22dc1a12c30557cab5b

@github-actions

github-actions Bot commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

✅ Gradle check result for 8bf11b2: SUCCESS

@codecov

codecov Bot commented Apr 7, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.23%. Comparing base (d26ad17) to head (8bf11b2).
⚠️ Report is 15 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main   #21153      +/-   ##
============================================
- Coverage     73.35%   73.23%   -0.13%     
+ Complexity    73209    73148      -61     
============================================
  Files          5921     5932      +11     
  Lines        333798   333924     +126     
  Branches      48124    48125       +1     
============================================
- Hits         244862   244549     -313     
- Misses        69387    69830     +443     
+ Partials      19549    19545       -4     

☔ View full report in Codecov by Sentry.
📢 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 merged commit 9bfcc1d into opensearch-project:main Apr 7, 2026
36 of 38 checks passed
aparajita31pandey pushed a commit to aparajita31pandey/OpenSearch that referenced this pull request Apr 18, 2026
Signed-off-by: Andriy Redko <drreta@gmail.com>
Signed-off-by: Aparajita Pandey <aparajita31pandey@gmail.com>
pradeep-L pushed a commit to pradeep-L/OpenSearch that referenced this pull request Apr 21, 2026
Signed-off-by: Andriy Redko <drreta@gmail.com>
rithinpullela added a commit to rithinpullela/ml-commons that referenced this pull request May 5, 2026
OpenSearch build-tools' GlobalBuildInfoPlugin now requires Gradle 9.4.1+,
causing every CI run (including main) to fail during project evaluation
with "Gradle 9.4.1+ is required".

Mirrors opensearch-project/OpenSearch#21153.

Signed-off-by: rithin-pullela-aws <rithinp@amazon.com>
rishabh6788 pushed a commit to opensearch-project/ml-commons that referenced this pull request May 5, 2026
OpenSearch build-tools' GlobalBuildInfoPlugin now requires Gradle 9.4.1+,
causing every CI run (including main) to fail during project evaluation
with "Gradle 9.4.1+ is required".

Mirrors opensearch-project/OpenSearch#21153.

Signed-off-by: rithin-pullela-aws <rithinp@amazon.com>
dai-chen added a commit to dai-chen/sql-1 that referenced this pull request May 7, 2026
Bump Gradle wrapper from 9.2.0 to 9.4.1 to satisfy the minimum version
required by opensearch-build-tools 3.7.0-SNAPSHOT (GlobalBuildInfoPlugin).

Exclude 24 @Ignore-annotated test classes from integTest at the Gradle
layer to work around a Gradle 9.4.1+ bug where
TestEventReporterAsListener crashes with ClassCastException when
encountering class-level @ignore annotations. These classes were already
skipped by JUnit; this moves the skip earlier to avoid the buggy bridge.
Not fixed as of Gradle 9.5.0.

Refs: opensearch-project/OpenSearch#21153, opensearch-project#5406
Signed-off-by: Chen Dai <daichen@amazon.com>
This was referenced Aug 3, 2026
This was referenced Aug 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Build Libraries & Interfaces 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