Skip to content

Commit baff726

Browse files
authored
Extend timeout in IngestGeoIpClientYamlTestSuiteIT (#138610) (#138645)
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 4fb85c6) # Conflicts: # modules/ingest-geoip/src/yamlRestTest/java/org/elasticsearch/ingest/geoip/IngestGeoIpClientYamlTestSuiteIT.java # muted-tests.yml
1 parent d9dc935 commit baff726

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

modules/ingest-geoip/src/yamlRestTest/java/org/elasticsearch/ingest/geoip/IngestGeoIpClientYamlTestSuiteIT.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@
3232
import java.io.IOException;
3333
import java.util.List;
3434
import java.util.Map;
35+
import java.util.concurrent.TimeUnit;
3536

3637
import static org.hamcrest.Matchers.containsInAnyOrder;
3738
import static org.hamcrest.Matchers.equalTo;
@@ -92,7 +93,9 @@ public void waitForDatabases() throws Exception {
9293
databaseNames,
9394
containsInAnyOrder("GeoLite2-City.mmdb", "GeoLite2-Country.mmdb", "GeoLite2-ASN.mmdb", "MyCustomGeoLite2-City.mmdb")
9495
);
95-
});
96+
// Downloading all four databases may take some time, so we set a longer timeout here.
97+
// If 20 seconds prove insufficient, we should first investigate whether we can speed up the database downloader.
98+
}, 20, TimeUnit.SECONDS);
9699
}
97100

98101
/**

0 commit comments

Comments
 (0)