Skip to content

Conversation

@gaobinlong
Copy link
Contributor

Description

The skip version in 400_max_score.yml is not correct, that causes the tests are skipped, this PR fixes the version and also fixes the test failure.

Related Issues

No issue.

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.

@gaobinlong gaobinlong requested a review from a team as a code owner August 13, 2025 07:11
@github-actions
Copy link
Contributor

✅ Gradle check result for 4460199: SUCCESS

@codecov
Copy link

codecov bot commented Aug 13, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.85%. Comparing base (7ff34ac) to head (4460199).
⚠️ Report is 7 commits behind head on 3.2.

Additional details and impacted files
@@             Coverage Diff              @@
##                3.2   #19050      +/-   ##
============================================
- Coverage     72.93%   72.85%   -0.08%     
+ Complexity    69352    69347       -5     
============================================
  Files          5643     5645       +2     
  Lines        318713   318779      +66     
  Branches      46112    46123      +11     
============================================
- Hits         232441   232249     -192     
- Misses        67488    67769     +281     
+ Partials      18784    18761      -23     

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

@gaobinlong
Copy link
Contributor Author

@andrross @cwperks mind taking a look at this PR? Thank you~

@andrross
Copy link
Member

Let's hold off on merging this until after the 3.2 release just to avoid any confusion around last minute changes. This is just a test fix so it's not needed for the release but it's fine to backport just so we have good test coverage on this branch in case there are any subsequent patch releases.

@andrross andrross merged commit 7d680c7 into opensearch-project:3.2 Aug 25, 2025
35 of 36 checks passed
mch2 pushed a commit to mch2/OpenSearch that referenced this pull request Sep 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants