Skip to content

Commit

Permalink
ovn-fake-multinode-tests: Fix log extraction.
Browse files Browse the repository at this point in the history
The paths were slightly wrong so no logs were retrieved/printed.

Fixes: 6c6a7ad ("Add fake multinode system tests.")
Signed-off-by: Dumitru Ceara <dceara@redhat.com>
Acked-by: Ales Musil <amusil@redhat.com>
Acked-by: Numan Siddique <numans@ovn.org>
  • Loading branch information
dceara committed May 8, 2023
1 parent 0b2d4c4 commit b2ea2ac
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/ovn-fake-multinode-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -172,7 +172,7 @@ jobs:
sudo podman exec -it ovn-chassis-1 ovs-vsctl show || :
sudo podman exec -it ovn-chassis-1 ip netns || :
sudo podman exec -it ovn-chassis-1 cat /var/log/ovn/ovn-controller.log || :
sudo cat tests/multinode-testsuite.dir/multinode-testsuite.log || :
sudo cat tests/multinode-testsuite.log || :
exit 1
fi
Expand All @@ -186,8 +186,8 @@ jobs:
mkdir logs
cp ovn/config.log ./logs/
# multinode tests are run as root, need to adjust permissions.
sudo chmod -R +r ovn/tests/multinode-testsuite.dir.* || true
cp -r ovn/tests/multinode-testsuite.dir.* ./logs/ || true
sudo chmod -R +r ovn/tests/multinode-testsuite.* || true
cp -r ovn/tests/multinode-testsuite.* ./logs/ || true
tar -czvf logs.tgz logs/
- name: upload logs on failure
Expand Down

0 comments on commit b2ea2ac

Please sign in to comment.