From a2099599ad87e993502ccc2d46a8a4f09eaa8625 Mon Sep 17 00:00:00 2001 From: Thomas Eizinger Date: Thu, 15 Feb 2024 12:41:50 +1100 Subject: [PATCH] ci: don't ping gateway before running perf (#3649) Running perf byitself should be enough to establish a connection, we don't need to explicitly do that before. Co-authored-by: Jamil --- .github/workflows/ci.yml | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 84c8c4e54..7ae9eb07d 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -235,19 +235,13 @@ jobs: include: - test_name: direct-perf setup: echo 'Noop' - execute: | - # Establish a channel - docker compose exec -it client timeout 60 \ - sh -c 'until ping -W 1 -c 1 172.20.0.100 &>/dev/null; do true; done' + execute: echo 'Noop' - test_name: relayed-perf setup: | # Disallow traffic between gateway and client container sudo iptables -I FORWARD 1 -s 172.28.0.100 -d 172.28.0.105 -j DROP sudo iptables -I FORWARD 1 -s 172.28.0.105 -d 172.28.0.100 -j DROP - execute: | - # Establish a channel - docker compose exec -it client timeout 60 \ - sh -c 'until ping -W 1 -c 1 172.20.0.100 &>/dev/null; do true; done' + execute: echo 'Noop' steps: - uses: actions/checkout@v4