Skip to content

Commit

Permalink
test/l4lb: use cilium-cli sysdump
Browse files Browse the repository at this point in the history
Use the `cilium sysdump` command instead of the deprecated
cilium-sysdump python script as we already do in other tests. This will
also include Hubble flows into the sysdump which the old sysdump tool
didn't.

Signed-off-by: Tobias Klauser <tobias@cilium.io>
  • Loading branch information
tklauser authored and nebril committed Oct 29, 2021
1 parent b71d528 commit 4973234
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions test/l4lb/test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -82,8 +82,12 @@ nsenter -t $(docker inspect kind-worker -f '{{ .State.Pid }}') -n /bin/sh -c \
'tc qdisc add dev eth0 clsact && tc filter add dev eth0 ingress bpf direct-action object-file ./test_tc_tunnel.o section decap'

collect_sysdump() {
curl -sLO https://github.com/cilium/cilium-sysdump/releases/latest/download/cilium-sysdump.zip
python cilium-sysdump.zip --output /tmp/cilium-sysdump-out
curl -sSL --remote-name-all https://github.com/cilium/cilium-cli/releases/latest/download/cilium-linux-amd64.tar.gz{,.sha256sum}
sha256sum --check cilium-linux-amd64.tar.gz.sha256sum
sudo tar xzvfC cilium-linux-amd64.tar.gz /usr/bin
rm cilium-linux-amd64.tar.gz{,.sha256sum}
cilium sysdump --output-filename cilium-sysdump-out
mv cilium-sysdump-out.zip /tmp
}

trap collect_sysdump ERR
Expand Down

0 comments on commit 4973234

Please sign in to comment.