Skip to content

Commit 0b7ba9b

Browse files
cbuescherarteam
andauthored
Backport Fix RetrySearchIntegTests (#122919) (#129404)
Don't simultaneously restart multiple nodes in a cluster. It causes data races when multiple primaries are trying to mark the `[[.snapshot-blob-cache][0]]` shard as stale. This backports #122919 to the 8.17 branch since we still get occasional test failures related to it there. Closes #127136 Co-authored-by: Artem Prigoda <[email protected]>
1 parent e7fc673 commit 0b7ba9b

File tree

1 file changed

+2
-0
lines changed
  • x-pack/plugin/searchable-snapshots/src/internalClusterTest/java/org/elasticsearch/xpack/searchablesnapshots

1 file changed

+2
-0
lines changed

x-pack/plugin/searchable-snapshots/src/internalClusterTest/java/org/elasticsearch/xpack/searchablesnapshots/RetrySearchIntegTests.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -90,6 +90,7 @@ public void testSearcherId() throws Exception {
9090
for (String allocatedNode : allocatedNodes) {
9191
if (randomBoolean()) {
9292
internalCluster().restartNode(allocatedNode);
93+
ensureGreen(indexName);
9394
}
9495
}
9596
ensureGreen(indexName);
@@ -151,6 +152,7 @@ public void testRetryPointInTime() throws Exception {
151152
final Set<String> allocatedNodes = internalCluster().nodesInclude(indexName);
152153
for (String allocatedNode : allocatedNodes) {
153154
internalCluster().restartNode(allocatedNode);
155+
ensureGreen(indexName);
154156
}
155157
ensureGreen(indexName);
156158
assertNoFailuresAndResponse(

0 commit comments

Comments
 (0)