Skip to content

Update Gradle to 9.4.1 and Jacoco to 0.8.14 - #4811

Merged
rishabh6788 merged 1 commit into
opensearch-project:mainfrom
rithinpullela:fix/gradle-wrapper-bump
May 5, 2026
Merged

Update Gradle to 9.4.1 and Jacoco to 0.8.14#4811
rishabh6788 merged 1 commit into
opensearch-project:mainfrom
rithinpullela:fix/gradle-wrapper-bump

Conversation

@rithinpullela

Copy link
Copy Markdown
Collaborator

Description

Bumps the Gradle wrapper from 9.2.0 → 9.4.1 and the Jacoco tool version from 0.8.13 → 0.8.14. This mirrors the change already landed in OpenSearch core: opensearch-project/OpenSearch#21153 (merged 2026-04-07).

Why

OpenSearch's build-tools GlobalBuildInfoPlugin now requires Gradle 9.4.1+. Since ml-commons still pins the wrapper to 9.2.0, every CI run — including pushes to main — fails during project evaluation with:

* What went wrong:
A problem occurred evaluating project ':opensearch-ml-plugin'.
> Failed to apply plugin class 'org.opensearch.gradle.info.GlobalBuildInfoPlugin'.
   > Gradle 9.4.1+ is required

Failing CI examples

Precedence

OpenSearch core hit and fixed this exact issue one month ago in opensearch-project/OpenSearch#21153:

  • Same Gradle target (9.4.1) and same SHA-256.
  • Same Jacoco bump (0.8.13 → 0.8.14).
  • This PR's diff is byte-identical to core's, just applied to ml-commons' root build.gradle (where the jacoco.toolVersion lives) instead of core's gradle/code-coverage.gradle.

Sibling plugin repos (neural-search, k-NN) are still on 9.2.0 and will need the same bump.

Changes

  • gradle/wrapper/gradle-wrapper.properties: gradle-9.2.0-all.zipgradle-9.4.1-all.zip (+ updated SHA-256; line ordering matched to core).
  • build.gradle: jacoco.toolVersion = '0.8.13''0.8.14'.

Test plan

  • Ran ./gradlew spotlessCheck locally with the updated wrapper — BUILD SUCCESSFUL across all 7 modules.
  • CI spotless job goes green on this PR.
  • CI Build and Test job proceeds past project evaluation.

Issues resolved

Unblocks all failing CI runs on main and on open PRs caused by the Gradle version requirement.

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.

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>
@github-actions

github-actions Bot commented May 5, 2026

Copy link
Copy Markdown

PR Code Analyzer ❗

AI-powered 'Code-Diff-Analyzer' found issues on commit 5a6efea.

PathLineSeverityDescription
gradle/wrapper/gradle-wrapper.properties4highGradle wrapper distribution URL changed from 9.2.0 to 9.4.1. The SHA256 checksum was also replaced (old: 16f2b95838c1ddcf7242b1c39e7bbbb43c842f1f1a1a0dc4959b6d4d68abcac3, new: 708d2c6ecc97ca9a11838ef64a6c2301151b8dd10387e22dc1a12c30557cab5b). The Gradle wrapper is a privileged build bootstrapping mechanism — a tampered or substituted distribution could execute arbitrary code during any build. Maintainers must verify the new checksum matches the official Gradle 9.4.1 release at services.gradle.org.
build.gradle85highJaCoCo tool version bumped from 0.8.13 to 0.8.14. Per mandatory supply chain policy, all dependency version changes must be flagged. Maintainers should verify this version exists in the official JaCoCo release history and that the artifact hash matches the expected value before merging.

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.

@rithinpullela
rithinpullela temporarily deployed to ml-commons-cicd-env-require-approval May 5, 2026 16:44 — with GitHub Actions Inactive
@rithinpullela
rithinpullela had a problem deploying to ml-commons-cicd-env-require-approval May 5, 2026 16:44 — with GitHub Actions Error
@rithinpullela
rithinpullela had a problem deploying to ml-commons-cicd-env-require-approval May 5, 2026 16:44 — with GitHub Actions Failure
@rithinpullela
rithinpullela temporarily deployed to ml-commons-cicd-env-require-approval May 5, 2026 16:44 — with GitHub Actions Inactive
@codecov

codecov Bot commented May 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.42%. Comparing base (12f884e) to head (5a6efea).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #4811   +/-   ##
=========================================
  Coverage     77.42%   77.42%           
  Complexity    11907    11907           
=========================================
  Files           963      963           
  Lines         53326    53326           
  Branches       6503     6503           
=========================================
+ Hits          41285    41289    +4     
+ Misses         9289     9288    -1     
+ Partials       2752     2749    -3     
Flag Coverage Δ
ml-commons 77.42% <ø> (+<0.01%) ⬆️

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

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

@rithinpullela
rithinpullela temporarily deployed to ml-commons-cicd-env-require-approval May 5, 2026 17:45 — with GitHub Actions Inactive
@rithinpullela
rithinpullela temporarily deployed to ml-commons-cicd-env-require-approval May 5, 2026 17:45 — with GitHub Actions Inactive
@rishabh6788
rishabh6788 merged commit be86ad0 into opensearch-project:main May 5, 2026
16 of 21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants