Skip to content

Increment version to 3.10.0-SNAPSHOT - #5789

Closed
finnegancarroll wants to merge 6 commits into
opensearch-project:mainfrom
finnegancarroll:finnegancarroll/bump-opensearch-3.10
Closed

finnegancarroll wants to merge 6 commits into
opensearch-project:mainfrom
finnegancarroll:finnegancarroll/bump-opensearch-3.10

Conversation

@finnegancarroll

@finnegancarroll finnegancarroll commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

Description

Bump the default opensearch_version from 3.9.0-SNAPSHOT to 3.10.0-SNAPSHOT so the sql build tracks the current OpenSearch main line (main is 3.10.0).

Signed-off-by: Finn Carroll <carrofin@amazon.com>
@github-actions

github-actions Bot commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

PR Code Analyzer ❗

AI-powered 'Code-Diff-Analyzer' found issues on commit 7dc3288.

PathLineSeverityDescription
build.gradle9highBuild version bump from 3.9.0-SNAPSHOT to 3.10.0-SNAPSHOT. Per mandatory rule, all dependency/build configuration version changes must be flagged for maintainer verification, regardless of apparent legitimacy. Maintainers should confirm this version exists in the expected registry and aligns with the intended release cycle.

The table above displays the top 10 most important findings.

Total: 1 | Critical: 0 | High: 1 | 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: Finn Carroll <carrofin@amazon.com>
…napshots now published)

Signed-off-by: Finn Carroll <carrofin@amazon.com>
@finnegancarroll finnegancarroll added ci maintenance Improves code quality, but not the product labels Sep 24, 2026
The analytics-engine native library (libopensearch_native.so from the
feature-datafusion build) requires glibc 2.28 but the default al2 ci-runner
ships glibc 2.26, so the node fails at boot with 'Cannot open library'. Pin the
almalinux8 ci-runner (glibc 2.28) for the AE workflow.

Signed-off-by: Finn Carroll <carrofin@amazon.com>
Pass platform: almalinux8 to get-ci-image-tag so it returns the almalinux8
ci-runner image AND its matching start-command/options as a set. The default
al2 image (glibc 2.26) cannot dlopen the analytics-engine native library
(libopensearch_native.so requires glibc 2.28); almalinux8 ships glibc 2.28.

Supersedes the previous hardcoded-image attempt, which mismatched the al2
start-command (/node_al2 copy) against the almalinux8 image.

Signed-off-by: Finn Carroll <carrofin@amazon.com>
Signed-off-by: Finn Carroll <carrofin@amazon.com>
@codecov

codecov Bot commented Sep 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.23%. Comparing base (a29cf85) to head (7dc3288).
⚠️ Report is 3 commits behind head on main.

❌ Your project check has failed because the head coverage (63.23%) is below the target coverage (99.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #5789      +/-   ##
============================================
- Coverage     63.24%   63.23%   -0.01%     
- Complexity     8820     8823       +3     
============================================
  Files           938      938              
  Lines         40211    40236      +25     
  Branches       4530     4537       +7     
============================================
+ Hits          25432    25445      +13     
- Misses        13957    13966       +9     
- Partials        822      825       +3     
Flag Coverage Δ
sql-engine 63.23% <ø> (-0.01%) ⬇️

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

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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

Labels

ci maintenance Improves code quality, but not the product

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant