Skip to content

Commit c8836a8

Browse files
authored
[TEST] ensure cluster is stable before running testReindex (#122589)
Unable to reproduce however the coordinator node that is meant to route the write requests might've not been ready to do so in due time. This PR adds an ensureStableCluster in the test setup method. Fixes #120605
1 parent 1473b19 commit c8836a8

File tree

2 files changed

+1
-3
lines changed

2 files changed

+1
-3
lines changed

modules/reindex/src/internalClusterTest/java/org/elasticsearch/index/reindex/BulkByScrollUsesAllScrollDocumentsAfterConflictsIntegTests.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -97,6 +97,7 @@ public void setUpCluster() {
9797
// Use a single thread pool for writes so we can enforce a consistent ordering
9898
internalCluster().startDataOnlyNode(Settings.builder().put("thread_pool.write.size", 1).build());
9999
internalCluster().startCoordinatingOnlyNode(Settings.EMPTY);
100+
ensureStableCluster(3);
100101
}
101102

102103
public void testUpdateByQuery() throws Exception {

muted-tests.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -187,9 +187,6 @@ tests:
187187
issue: https://github.com/elastic/elasticsearch/issues/120482
188188
- class: org.elasticsearch.xpack.sql.expression.function.scalar.datetime.DateTimeToCharProcessorTests
189189
issue: https://github.com/elastic/elasticsearch/issues/120575
190-
- class: org.elasticsearch.index.reindex.BulkByScrollUsesAllScrollDocumentsAfterConflictsIntegTests
191-
method: testReindex
192-
issue: https://github.com/elastic/elasticsearch/issues/120605
193190
- class: org.elasticsearch.xpack.inference.DefaultEndPointsIT
194191
method: testMultipleInferencesTriggeringDownloadAndDeploy
195192
issue: https://github.com/elastic/elasticsearch/issues/120668

0 commit comments

Comments
 (0)