-
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 transient failure in UT testIndicesRoutingDiffWhenIndexDeletedAnd… #15592
base: main
Are you sure you want to change the base?
Fix transient failure in UT testIndicesRoutingDiffWhenIndexDeletedAnd… #15592
Conversation
❌ Gradle check result for a5aad7d: 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? |
Signed-off-by: Shailendra Singh <[email protected]>
a5aad7d
to
4a2cb8e
Compare
@@ -526,7 +526,7 @@ public void testIndicesRoutingDiffWhenIndexDeletedAndAdded() { | |||
String indexNameToDelete = allIndices.get(randomIntBetween(0, allIndices.size() - 1)); | |||
RoutingTable.Builder updatedRoutingTableBuilder = RoutingTable.builder(routingTable).remove(indexNameToDelete); | |||
|
|||
String indexName = randomAlphaOfLength(randomIntBetween(1, 50)); | |||
String indexName = randomAlphaOfLength(randomIntBetween(51, 100)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
how does this help fix the flakiness?
❌ Gradle check result for 4a2cb8e: 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? |
This PR is stalled because it has been open for 30 days with no activity. |
Description
Fix transient failure in UT testIndicesRoutingDiffWhenIndexDeletedAndAdd
Related Issues
Resolves #15588
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.