-
Notifications
You must be signed in to change notification settings - Fork 1.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix Flaky Test RecoveryFromGatewayIT.testShardStoreFetchMultiNodeMultiIndexesUsingBatchAction #16093
Conversation
0c7405e
to
c29a7c1
Compare
❌ Gradle check result for c29a7c1: 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? |
❌ Gradle check result for 250a33b: 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? |
❌ Gradle check result for 99a1b04: 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? |
ba7d90c
to
c6058a6
Compare
❌ Gradle check result for c6058a6: 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? |
server/src/internalClusterTest/java/org/opensearch/gateway/RecoveryFromGatewayIT.java
Show resolved
Hide resolved
❌ Gradle check result for c6058a6: 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? |
❌ Gradle check result for cee67fe: 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? |
❌ Gradle check result for cee67fe: 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? |
…StoreFetchMultiNodeMultiIndexesUsingBatchAction Signed-off-by: kkewwei <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #16093 +/- ##
============================================
- Coverage 71.97% 71.90% -0.08%
+ Complexity 64771 64707 -64
============================================
Files 5302 5302
Lines 302473 302473
Branches 43699 43699
============================================
- Hits 217715 217498 -217
- Misses 66950 67150 +200
- Partials 17808 17825 +17 ☔ View full report in Codecov by Sentry. |
@gaobinlong it's ok now, please have a look. |
…StoreFetchMultiNodeMultiIndexesUsingBatchAction (opensearch-project#16093) Signed-off-by: kkewwei <[email protected]>
…StoreFetchMultiNodeMultiIndexesUsingBatchAction (opensearch-project#16093) Signed-off-by: kkewwei <[email protected]>
…StoreFetchMultiNodeMultiIndexesUsingBatchAction (opensearch-project#16093) Signed-off-by: kkewwei <[email protected]>
Description
In RecoveryFromGatewayIT test, the created indices may be not started yet, but is used immediately, which will cause unexpected exception. when it throws the exception, the scene is as follows:
Related Issues
Resolves #14304
Check List
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.