Skip to content

Update to almalinux:10#20482

Merged
andrross merged 4 commits intoopensearch-project:mainfrom
prudhvigodithi:gradle-check
Jan 26, 2026
Merged

Update to almalinux:10#20482
andrross merged 4 commits intoopensearch-project:mainfrom
prudhvigodithi:gradle-check

Conversation

@prudhvigodithi
Copy link
Copy Markdown
Member

@prudhvigodithi prudhvigodithi commented Jan 26, 2026

Description

To fix the following CI errors with almalinux:8. This issue is failing distribution:docker:buildArm64DockerImage with almalinux:8 docker image for assemble.

image

Can see the above failure on the following PR's:

Related Issues

Resolves #[Issue number to be closed when this PR is merged]

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: Prudhvi Godithi <pgodithi@amazon.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Jan 26, 2026

📝 Walkthrough

Walkthrough

Updates the AlmaLinux Docker base image version from 8 to 9 in the build configuration. This is a single-line change to an enum constant that specifies the container image reference used during the build process.

Changes

Cohort / File(s) Summary
Docker Base Image Version
buildSrc/src/main/java/org/opensearch/gradle/DockerBase.java
Updated ALMALINUX enum constant from "almalinux:8" to "almalinux:9". Changes the Docker base image tag without modifying any logic or structure.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title states 'Update to almalinux:10' but the actual change in the code updates ALMALINUX from 'almalinux:8' to 'almalinux:9', not 'almalinux:10'. The title is misleading and does not accurately reflect the changeset. Correct the PR title to 'Update to almalinux:9' to accurately reflect the version change being made in the code.
✅ Passed checks (2 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed The PR description provides a clear purpose (fixing almalinux:8 CI failures by upgrading to almalinux:9) with supporting evidence, but the Related Issues section is incomplete with only a placeholder.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Comment @coderabbitai help to get the list of available commands and usage tips.

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
@prudhvigodithi prudhvigodithi marked this pull request as ready for review January 26, 2026 19:04
@prudhvigodithi prudhvigodithi requested a review from a team as a code owner January 26, 2026 19:04
@prudhvigodithi
Copy link
Copy Markdown
Member Author

Updating to almalinux:9 does seem to resolve the issue. @cwperks @peterzhuamazon @andrross @reta WDYT?

Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
@prudhvigodithi prudhvigodithi changed the title Update to almalinux:9 Update to almalinux:10 Jan 26, 2026
@github-actions
Copy link
Copy Markdown
Contributor

❕ Gradle check result for a2ac500: UNSTABLE

Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 26, 2026

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 73.27%. Comparing base (672039d) to head (a2ac500).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...rc/main/java/org/opensearch/gradle/DockerBase.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main   #20482      +/-   ##
============================================
+ Coverage     73.25%   73.27%   +0.02%     
+ Complexity    71979    71964      -15     
============================================
  Files          5796     5796              
  Lines        329287   329287              
  Branches      47419    47419              
============================================
+ Hits         241203   241290      +87     
+ Misses        68759    68638     -121     
- Partials      19325    19359      +34     

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

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Performance Roadmap Jan 26, 2026
@andrross andrross merged commit eeec4d9 into opensearch-project:main Jan 26, 2026
34 of 35 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Performance Roadmap Jan 26, 2026
tanyabti pushed a commit to tanyabti/OpenSearch that referenced this pull request Feb 24, 2026
Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
tanyabti pushed a commit to tanyabti/OpenSearch that referenced this pull request Feb 24, 2026
Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants