Skip to content

Commit cb3840a

Browse files
authored
[TEST] ensure cluster is stable before running testReindex (#122589) (#122621)
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 cc8404a commit cb3840a

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
@@ -196,9 +196,6 @@ tests:
196196
issue: https://github.com/elastic/elasticsearch/issues/120482
197197
- class: org.elasticsearch.xpack.sql.expression.function.scalar.datetime.DateTimeToCharProcessorTests
198198
issue: https://github.com/elastic/elasticsearch/issues/120575
199-
- class: org.elasticsearch.index.reindex.BulkByScrollUsesAllScrollDocumentsAfterConflictsIntegTests
200-
method: testReindex
201-
issue: https://github.com/elastic/elasticsearch/issues/120605
202199
- class: org.elasticsearch.xpack.inference.DefaultEndPointsIT
203200
method: testMultipleInferencesTriggeringDownloadAndDeploy
204201
issue: https://github.com/elastic/elasticsearch/issues/120668

0 commit comments

Comments
 (0)