The test failure occurs quite consistently (~50% of the runs) when using
a musl static build. It always happens if I add `sync` or `sleep` before
vswitchd shutdown. The warning message can be seen in the
retained vswitchd log (-d) even when the test case passes.

I believe a race condition between vswitchd flushing the log to disc and
the test code reading from the log file explains why the test case
doesn't fail consistently.

Signed-off-by: Ihar Hrachyshka <[email protected]>
---
 tests/tunnel.at | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/tunnel.at b/tests/tunnel.at
index 24219350a..51c486681 100644
--- a/tests/tunnel.at
+++ b/tests/tunnel.at
@@ -129,7 +129,7 @@ AT_CHECK([ovs-appctl dpctl/dump-flows | tail -1], [0], [dnl
 
recirc_id(0),tunnel(src=3.3.3.200/255.255.255.0,dst=1.1.1.1,ttl=64,tp_src=1,tp_dst=123),in_port(1),eth(),eth_type(0x0800),
 packets:0, bytes:0, used:never, actions:2
 ])
 
-OVS_VSWITCHD_STOP
+OVS_VSWITCHD_STOP(["/receive tunnel port not found/d"])
 AT_CLEANUP
 
 AT_SETUP([tunnel - too long nested attributes])
-- 
2.50.1

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to