Skip to content

Commit 201ba70

Browse files
ralfliciordex
authored andcommitted
selftests: ovpn: reduce ping count in test.sh
The second stage of test.sh ("run baseline data traffic") performs a basic connectivity check with ping -qfc 500 -w 3. On slower CI instances this is too strict for TCP: the RTT is high enough that 500 echo requests do not reliably complete within 3 seconds, so the stage flakes and the test fails even though the ovpn setup is healthy. Reduce the packet count to 100 for both the plain and 3000-byte pings in that stage. This still verifies peer setup, key exchange, routing, and data-path traffic, without making the basic connectivity check depend on timing out under load. Fixes: 959bc33 ("testing/selftests: add test tool and scripts for ovpn module") Signed-off-by: Ralf Lici <ralf@mandelbit.com> Signed-off-by: Antonio Quartulli <antonio@openvpn.net>
1 parent c539cb3 commit 201ba70

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

  • tools/testing/selftests/net/ovpn

tools/testing/selftests/net/ovpn/test.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,10 +98,10 @@ ovpn_run_basic_traffic() {
9898
sleep 0.3
9999
ovpn_cmd_ok "send baseline traffic to peer ${p}" \
100100
ip netns exec ovpn_peer0 \
101-
ping -qfc 500 -w 3 5.5.5.$((p + 1))
101+
ping -qfc 100 -w 3 5.5.5.$((p + 1))
102102
ovpn_cmd_ok "send large-payload traffic to peer ${p}" \
103103
ip netns exec ovpn_peer0 \
104-
ping -qfc 500 -s 3000 -w 3 5.5.5.$((p + 1))
104+
ping -qfc 100 -s 3000 -w 3 5.5.5.$((p + 1))
105105

106106
wait "${tcpdump_pid1}" || return 1
107107
wait "${tcpdump_pid2}" || return 1

0 commit comments

Comments
 (0)