From 6d25f5909e148371e35c07a9022f2179d4fd7476 Mon Sep 17 00:00:00 2001
From: Niels Bauman <33722607+nielsbauman@users.noreply.github.com>
Date: Wed, 26 Nov 2025 10:32:12 +0100
Subject: [PATCH] Extend timeout in `IngestGeoIpClientYamlTestSuiteIT`
 (#138610)

Some slow CI test clusters are not able to download all four GeoIP databases within 10 seconds, so we bump the timeout to 20 seconds.

See #138460 for a more detailed analysis.

Fixes #138460
Fixes #138502

(cherry picked from commit 4fb85c605839cadf0f0bdcb95834cb9d296f5e72)

# Conflicts:
#	muted-tests.yml
---
 .../ingest/geoip/IngestGeoIpClientYamlTestSuiteIT.java       | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

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..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
@@ -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,9 @@ 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);
     }
 
     @SuppressForbidden(reason = "fixtures use java.io.File based APIs")