-
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 tests in org.opensearch.cluster.routing.remote.RemoteRoutingTableServiceTests #14264
Conversation
…gTableServiceTests Signed-off-by: Gao Binlong <gbinlong@amazon.com>
❌ Gradle check result for c1ffe62: 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? |
Known issue:#12788 |
❌ Gradle check result for 9415706: 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 9415706: 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? |
Known issue:#1703 |
❌ Gradle check result for b5f2a77: 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 73a8470: 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 73a8470: 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 #14264 +/- ##
============================================
+ Coverage 71.42% 71.61% +0.19%
- Complexity 59978 62023 +2045
============================================
Files 4985 5118 +133
Lines 282275 291829 +9554
Branches 40946 42188 +1242
============================================
+ Hits 201603 208983 +7380
- Misses 63999 65583 +1584
- Partials 16673 17263 +590 ☔ View full report in Codecov by Sentry. |
…gTableServiceTests (opensearch-project#14264) Signed-off-by: Gao Binlong <gbinlong@amazon.com>
…gTableServiceTests (opensearch-project#14264) (opensearch-project#14401) (cherry picked from commit 5ec2abe) Signed-off-by: Gao Binlong <gbinlong@amazon.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Signed-off-by: kkewwei <kkewwei@163.com>
…gTableServiceTests (opensearch-project#14264) Signed-off-by: Gao Binlong <gbinlong@amazon.com>
Description
This PR fixes the flaky test cases in org.opensearch.cluster.routing.remote.RemoteRoutingTableServiceTests, the failure is caused by that
randomInt(1000)
can generate0
but 0 is not valid for the index settingindex.number_of_shards
.By the way, some unused code in this test file are also deleted.
Changelog is not needed, backport to 2.x is needed.
Related Issues
#14263
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.