From 43af0ab1a20f3efde6cce6c6787bc8dbcdd1d124 Mon Sep 17 00:00:00 2001 From: Markus Pettersson Date: Fri, 8 Nov 2024 12:58:27 +0100 Subject: [PATCH] Mitigate `test_multihop` flakiness Limit relay selection in `test_multihop` to reduce flakiness. Hopefully this should be able to (at least partially) mitigate timeout-related issues. --- test/test-manager/src/tests/tunnel.rs | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/test/test-manager/src/tests/tunnel.rs b/test/test-manager/src/tests/tunnel.rs index e7056a629d4e..beaeb63ca54a 100644 --- a/test/test-manager/src/tests/tunnel.rs +++ b/test/test-manager/src/tests/tunnel.rs @@ -294,7 +294,16 @@ pub async fn test_multihop( rpc: ServiceClient, mut mullvad_client: MullvadProxyClient, ) -> Result<(), Error> { - let query = RelayQueryBuilder::new().wireguard().multihop().build(); + // NOTE: We have experienced flakiness due to timeout issues if distant relays are selected. + // This is an attempt to try to reduce this type of flakiness. + use helpers::custom_lists::LowLatency; + + let query = RelayQueryBuilder::new() + .wireguard() + .multihop() + .location(LowLatency) + .entry(LowLatency) + .build(); apply_settings_from_relay_query(&mut mullvad_client, query).await?;