Skip to content

Commit c7978a3

Browse files
committed
Fix data stream retrieval in TimeSeriesDataStreamsIT
This test had the potential to fail when two consecutive GET data streams requests would hit two different nodes, where one node already had the cluster state that contained the new backing index and the other node didn't yet. Caused by elastic#122852 Fixes elastic#124987
1 parent a6f685c commit c7978a3

File tree

2 files changed

+0
-6
lines changed

2 files changed

+0
-6
lines changed

muted-tests.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -363,9 +363,6 @@ tests:
363363
- class: org.elasticsearch.packaging.test.BootstrapCheckTests
364364
method: test10Install
365365
issue: https://github.com/elastic/elasticsearch/issues/124957
366-
- class: org.elasticsearch.xpack.ilm.TimeSeriesDataStreamsIT
367-
method: testRolloverAction
368-
issue: https://github.com/elastic/elasticsearch/issues/124987
369366
- class: org.elasticsearch.packaging.test.DockerTests
370367
method: test011SecurityEnabledStatus
371368
issue: https://github.com/elastic/elasticsearch/issues/124990

x-pack/plugin/ilm/qa/multi-node/src/javaRestTest/java/org/elasticsearch/xpack/ilm/TimeSeriesDataStreamsIT.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -83,9 +83,6 @@ public void testRolloverAction() throws Exception {
8383
final var backingIndices = getDataStreamBackingIndexNames(dataStream);
8484
assertEquals(2, backingIndices.size());
8585
assertTrue(Boolean.parseBoolean((String) getIndexSettingsAsMap(backingIndices.getLast()).get("index.hidden")));
86-
});
87-
assertBusy(() -> {
88-
final var backingIndices = getDataStreamBackingIndexNames(dataStream);
8986
assertEquals(PhaseCompleteStep.finalStep("hot").getKey(), getStepKeyForIndex(client(), backingIndices.getFirst()));
9087
});
9188
}

0 commit comments

Comments
 (0)