Skip to content

Commit f831114

Browse files
nielsbaumanomricohenn
authored andcommitted
Fix data stream retrieval in TimeSeriesDataStreamsIT (elastic#125621)
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 014f5ef commit f831114

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
@@ -357,9 +357,6 @@ tests:
357357
- class: org.elasticsearch.packaging.test.BootstrapCheckTests
358358
method: test10Install
359359
issue: https://github.com/elastic/elasticsearch/issues/124957
360-
- class: org.elasticsearch.xpack.ilm.TimeSeriesDataStreamsIT
361-
method: testRolloverAction
362-
issue: https://github.com/elastic/elasticsearch/issues/124987
363360
- class: org.elasticsearch.packaging.test.DockerTests
364361
method: test011SecurityEnabledStatus
365362
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)