Skip to content

Commit c5b2973

Browse files
authored
Unmute the test and add logging (#132609)
1 parent 7bd95dc commit c5b2973

File tree

2 files changed

+4
-8
lines changed

2 files changed

+4
-8
lines changed

muted-tests.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -301,9 +301,6 @@ tests:
301301
- class: org.elasticsearch.packaging.test.TemporaryDirectoryConfigTests
302302
method: test21AcceptsCustomPathInDocker
303303
issue: https://github.com/elastic/elasticsearch/issues/128114
304-
- class: org.elasticsearch.xpack.search.CrossClusterAsyncSearchIT
305-
method: testCCSClusterDetailsWhereAllShardsSkippedInCanMatch
306-
issue: https://github.com/elastic/elasticsearch/issues/128418
307304
- class: org.elasticsearch.xpack.esql.plugin.DataNodeRequestSenderIT
308305
method: testSearchWhileRelocating
309306
issue: https://github.com/elastic/elasticsearch/issues/128500

server/src/internalClusterTest/java/org/elasticsearch/search/ccs/CrossClusterSearchIT.java

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -194,6 +194,9 @@ public void testCCSClusterDetailsWhereAllShardsSkippedInCanMatch() throws Except
194194
assertNotNull(response);
195195

196196
Clusters clusters = response.getClusters();
197+
if (clusters.hasPartialResults()) {
198+
logger.info("Unexpected partial results for search: {}", response);
199+
}
197200
assertFalse("search cluster results should NOT be marked as partial", clusters.hasPartialResults());
198201
assertThat(clusters.getTotal(), equalTo(2));
199202
assertThat(clusters.getClusterStateCount(Cluster.Status.SUCCESSFUL), equalTo(2));
@@ -223,11 +226,7 @@ public void testCCSClusterDetailsWhereAllShardsSkippedInCanMatch() throws Except
223226
assertThat(remoteClusterSearchInfo.getStatus(), equalTo(Cluster.Status.SUCCESSFUL));
224227
assertThat(remoteClusterSearchInfo.getTotalShards(), equalTo(remoteNumShards));
225228
assertThat(remoteClusterSearchInfo.getSuccessfulShards(), equalTo(remoteNumShards));
226-
if (clusters.isCcsMinimizeRoundtrips()) {
227-
assertThat(remoteClusterSearchInfo.getSkippedShards(), equalTo(remoteNumShards));
228-
} else {
229-
assertThat(remoteClusterSearchInfo.getSkippedShards(), equalTo(remoteNumShards));
230-
}
229+
assertThat(remoteClusterSearchInfo.getSkippedShards(), equalTo(remoteNumShards));
231230
assertThat(remoteClusterSearchInfo.getFailedShards(), equalTo(0));
232231
assertThat(remoteClusterSearchInfo.getFailures().size(), equalTo(0));
233232
assertThat(remoteClusterSearchInfo.getTook().millis(), greaterThanOrEqualTo(0L));

0 commit comments

Comments
 (0)