diff mbox series

[ovs-dev,v2,1/2] system-ovn.at: Fix flaky "load-balancing" test.

Message ID 9ec9642c0db469f04d6b52b28debe5b16f23753a.camel@redhat.com
State Superseded
Headers show
Series [ovs-dev,v2,1/2] system-ovn.at: Fix flaky "load-balancing" test. | expand

Checks

Context Check Description
ovsrobot/apply-robot success apply and check: success
ovsrobot/github-robot-_Build_and_Test fail github build: failed
ovsrobot/github-robot-_ovn-kubernetes fail github build: failed

Commit Message

Xavier Simonart Aug. 4, 2021, 6:26 a.m. UTC
When running load-balancing test, tcpdump is executed in background.
When the backround tcpdump stops, it sends to stderr a message
(such as number of packets captured) which might be catched by the
following command (wget) causing the test to fail.

Fixes: ebbcd8e8cc ("northd: add reject action for lb with no backends")
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=1989517
Signed-off-by: Xavier Simonart <xsimonar@redhat.com>
---
v1 -> v2
* Address the fact that tcpdump sends to stderr
---
 tests/system-ovn.at | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--
2.31.1
diff mbox series

Patch

diff --git a/tests/system-ovn.at b/tests/system-ovn.at
index 4288d80e5..2d21fb262 100644
--- a/tests/system-ovn.at
+++ b/tests/system-ovn.at
@@ -1593,7 +1593,7 @@  OVS_WAIT_UNTIL([
 ovn-nbctl --reject lb-add lb3 30.0.0.10:80 ""
 ovn-nbctl ls-lb-add foo lb3
 # Filter reset segments
-NS_CHECK_EXEC([foo1], [tcpdump -c 1 -neei foo1 ip[[33:1]]=0x14 > rst.pcap &])
+NS_CHECK_EXEC([foo1], [tcpdump -c 1 -neei foo1 ip[[33:1]]=0x14 > rst.pcap 2>/dev/null &])
 sleep 1
 NS_CHECK_EXEC([foo1], [wget -q 30.0.0.10],[4])