From 85d54e4868e64bc96d3c16cdf919a90e7c8ee624 Mon Sep 17 00:00:00 2001 From: Niels Bauman Date: Tue, 25 Nov 2025 17:27:49 +0100 Subject: [PATCH 1/2] Extend timeout in `IngestGeoIpClientYamlTestSuiteIT` --- .../ingest/geoip/IngestGeoIpClientYamlTestSuiteIT.java | 4 +++- muted-tests.yml | 3 --- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/modules/ingest-geoip/src/yamlRestTest/java/org/elasticsearch/ingest/geoip/IngestGeoIpClientYamlTestSuiteIT.java b/modules/ingest-geoip/src/yamlRestTest/java/org/elasticsearch/ingest/geoip/IngestGeoIpClientYamlTestSuiteIT.java index 69bc58559bede..6639040ef3ee7 100644 --- a/modules/ingest-geoip/src/yamlRestTest/java/org/elasticsearch/ingest/geoip/IngestGeoIpClientYamlTestSuiteIT.java +++ b/modules/ingest-geoip/src/yamlRestTest/java/org/elasticsearch/ingest/geoip/IngestGeoIpClientYamlTestSuiteIT.java @@ -38,6 +38,7 @@ import java.util.List; import java.util.Map; import java.util.Objects; +import java.util.concurrent.TimeUnit; import static org.hamcrest.Matchers.containsInAnyOrder; import static org.hamcrest.Matchers.equalTo; @@ -158,7 +159,8 @@ static void assertDatabasesLoaded() throws Exception { // ensure that the extra config database has been set up, too: assertThat(node.get("config_databases"), equalTo(List.of("asn.mmdb"))); - }); + // Downloading all four databases may take some time, so we set a longer timeout here. + }, 20, TimeUnit.SECONDS); } @SuppressForbidden(reason = "fixtures use java.io.File based APIs") diff --git a/muted-tests.yml b/muted-tests.yml index cc37970f53fd0..fbe9c1f552f2a 100644 --- a/muted-tests.yml +++ b/muted-tests.yml @@ -435,9 +435,6 @@ tests: - class: org.elasticsearch.xpack.inference.integration.AuthorizationTaskExecutorIT method: testCreatesEisChatCompletion_DoesNotRemoveEndpointWhenNoLongerAuthorized issue: https://github.com/elastic/elasticsearch/issues/138480 -- class: org.elasticsearch.ingest.geoip.IngestGeoIpClientYamlTestSuiteIT - method: test {yaml=ingest_geoip/60_ip_location_databases/Test adding, getting, and removing ip location databases} - issue: https://github.com/elastic/elasticsearch/issues/138502 - class: org.elasticsearch.xpack.exponentialhistogram.ExponentialHistogramFieldMapperTests method: testFormattedDocValues issue: https://github.com/elastic/elasticsearch/issues/138504 From 2da145168535b0c8d23457db618f952e89f46e0a Mon Sep 17 00:00:00 2001 From: Niels Bauman Date: Tue, 25 Nov 2025 18:09:52 +0100 Subject: [PATCH 2/2] Add comment --- .../ingest/geoip/IngestGeoIpClientYamlTestSuiteIT.java | 1 + 1 file changed, 1 insertion(+) diff --git a/modules/ingest-geoip/src/yamlRestTest/java/org/elasticsearch/ingest/geoip/IngestGeoIpClientYamlTestSuiteIT.java b/modules/ingest-geoip/src/yamlRestTest/java/org/elasticsearch/ingest/geoip/IngestGeoIpClientYamlTestSuiteIT.java index 6639040ef3ee7..f305f8852ffad 100644 --- a/modules/ingest-geoip/src/yamlRestTest/java/org/elasticsearch/ingest/geoip/IngestGeoIpClientYamlTestSuiteIT.java +++ b/modules/ingest-geoip/src/yamlRestTest/java/org/elasticsearch/ingest/geoip/IngestGeoIpClientYamlTestSuiteIT.java @@ -160,6 +160,7 @@ static void assertDatabasesLoaded() throws Exception { // ensure that the extra config database has been set up, too: assertThat(node.get("config_databases"), equalTo(List.of("asn.mmdb"))); // Downloading all four databases may take some time, so we set a longer timeout here. + // If 20 seconds prove insufficient, we should first investigate whether we can speed up the database downloader. }, 20, TimeUnit.SECONDS); }