-
Notifications
You must be signed in to change notification settings - Fork 2k
Fix flaky test RemoteIndexRecoveryIT.testRerouteRecovery #17228
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 RemoteIndexRecoveryIT.testRerouteRecovery #17228
Conversation
Signed-off-by: Sachin Kale <[email protected]>
❌ Gradle check result for 3a2e2f3: 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 3a2e2f3: 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? |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #17228 +/- ##
============================================
+ Coverage 72.24% 72.29% +0.05%
+ Complexity 65704 65660 -44
============================================
Files 5318 5318
Lines 305674 305674
Branches 44349 44349
============================================
+ Hits 220834 220991 +157
+ Misses 66769 66521 -248
- Partials 18071 18162 +91 ☔ View full report in Codecov by Sentry. |
…project#17228) Signed-off-by: Sachin Kale <[email protected]>
…project#17228) Signed-off-by: Sachin Kale <[email protected]> Signed-off-by: Vinay Krishna Pudyodu <[email protected]>
Description
IndexRecoveryIT.testRerouteRecovery
triggers shard movement and verifies few details while recovery is in progress. It also slows down recovery to make sure that shard is not moved before we assert.RemoteIndexRecoveryIT extends
IndexRecoveryIT`, we reduced number of docs to be ingested by 5x (this is to decrease test run time as latency of ingestion is higher with remote store enabled).OpenSearch/server/src/internalClusterTest/java/org/opensearch/remotestore/RemoteIndexRecoveryIT.java
Lines 67 to 70 in c06f53e
Related Issues
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.