فهرست منبع

Fix testDoesNotResolveClosedIndex (#133302)

As commented, the index must have all replica shards assigned before closing.
Create index and prior indexing only guarantee primary shard is assigned.
Ievgen Degtiarenko 1 ماه پیش
والد
کامیت
a79ea5daee

+ 3 - 5
x-pack/plugin/esql/src/internalClusterTest/java/org/elasticsearch/xpack/esql/plugin/IndexResolutionIT.java

@@ -19,7 +19,6 @@ import org.elasticsearch.common.util.CollectionUtils;
 import org.elasticsearch.datastreams.DataStreamsPlugin;
 import org.elasticsearch.index.IndexNotFoundException;
 import org.elasticsearch.plugins.Plugin;
-import org.elasticsearch.test.junit.annotations.TestIssueLogging;
 import org.elasticsearch.xpack.esql.VerificationException;
 import org.elasticsearch.xpack.esql.action.AbstractEsqlIntegTestCase;
 import org.elasticsearch.xpack.esql.action.EsqlQueryResponse;
@@ -110,13 +109,12 @@ public class IndexResolutionIT extends AbstractEsqlIntegTestCase {
         );
     }
 
-    @TestIssueLogging(
-        value = "org.elasticsearch.cluster.metadata.MetadataIndexStateService:DEBUG",
-        issueUrl = "https://github.com/elastic/elasticsearch/issues/133011"
-    )
     public void testDoesNotResolveClosedIndex() {
         assertAcked(client().admin().indices().prepareCreate("index-1"));
         indexRandom(true, "index-1", 10);
+        // Create index only waits for primary/indexing shard to be assigned.
+        // This is enough to index and search documents, however all shards (including replicas) must be assigned before close.
+        ensureGreen("index-1");
         assertAcked(client().admin().indices().prepareClose("index-1"));
         assertAcked(client().admin().indices().prepareCreate("index-2"));
         indexRandom(true, "index-2", 15);