Skip to content

Conversation

@opensearch-trigger-bot
Copy link
Contributor

Backport 8ee5eeb from #17576.

* add validation check for SBP service

Signed-off-by: Kaushal Kumar <ravi.kaushal97@gmail.com>

* add SBP should not handle the task tracking UT

Signed-off-by: Kaushal Kumar <ravi.kaushal97@gmail.com>

* add CHANGELOG entry

Signed-off-by: Kaushal Kumar <ravi.kaushal97@gmail.com>

* fix broken UT

Signed-off-by: Kaushal Kumar <ravi.kaushal97@gmail.com>

* address comments

Signed-off-by: Kaushal Kumar <ravi.kaushal97@gmail.com>

---------

Signed-off-by: Kaushal Kumar <ravi.kaushal97@gmail.com>
(cherry picked from commit 8ee5eeb)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Contributor

❌ Gradle check result for 9151b5f: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions
Copy link
Contributor

❕ Gradle check result for 9151b5f: UNSTABLE

  • TEST FAILURES:
      1 org.opensearch.gateway.RecoveryFromGatewayIT.testShardStoreFetchMultiNodeMultiIndexesUsingBatchAction

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

codecov bot commented Mar 17, 2025

Codecov Report

Attention: Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 71.98%. Comparing base (4cfa376) to head (9151b5f).
Report is 9 commits behind head on 2.x.

Files with missing lines Patch % Lines
...ain/java/org/opensearch/wlm/QueryGroupService.java 50.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##                2.x   #17591      +/-   ##
============================================
- Coverage     72.13%   71.98%   -0.15%     
+ Complexity    66109    65999     -110     
============================================
  Files          5342     5342              
  Lines        307285   307289       +4     
  Branches      44847    44848       +1     
============================================
- Hits         221666   221215     -451     
- Misses        67142    67588     +446     
- Partials      18477    18486       +9     

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

@jainankitk jainankitk merged commit c332506 into 2.x Mar 17, 2025
67 of 68 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant