Skip to content

Commit e96719d

Browse files
authored
Extend timeout in IngestGeoIpClientYamlTestSuiteIT (#138610) (#138642)
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: # muted-tests.yml
1 parent 5b62b70 commit e96719d

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
@@ -38,6 +38,7 @@
3838
import java.util.List;
3939
import java.util.Map;
4040
import java.util.Objects;
41+
import java.util.concurrent.TimeUnit;
4142

4243
import static org.hamcrest.Matchers.containsInAnyOrder;
4344
import static org.hamcrest.Matchers.equalTo;
@@ -158,7 +159,9 @@ static void assertDatabasesLoaded() throws Exception {
158159

159160
// ensure that the extra config database has been set up, too:
160161
assertThat(node.get("config_databases"), equalTo(List.of("asn.mmdb")));
161-
});
162+
// Downloading all four databases may take some time, so we set a longer timeout here.
163+
// If 20 seconds prove insufficient, we should first investigate whether we can speed up the database downloader.
164+
}, 20, TimeUnit.SECONDS);
162165
}
163166

164167
@SuppressForbidden(reason = "fixtures use java.io.File based APIs")

0 commit comments

Comments
 (0)