On 8/7/20 10:23 AM, [email protected] wrote: > From: Numan Siddique <[email protected]> > > The below system test is failing due to timing issues. > > 29. system-ovn.at:4491: testing ovn --Test packet drops due to incorrect > flows in physical table 33 ... > > This patch fixes it by making sure that the lports are up before sending an > ICMP packet. > > Signed-off-by: Numan Siddique <[email protected]> > --- > tests/system-ovn.at | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/tests/system-ovn.at b/tests/system-ovn.at > index 2af09b4d0b..d43739bd35 100644 > --- a/tests/system-ovn.at > +++ b/tests/system-ovn.at > @@ -4534,6 +4534,10 @@ ADD_NAMESPACES(sw0-p3-f) > ADD_VETH(sw0-p3-f, sw0-p3-f, br-int, "10.0.0.5/24", "10:54:00:00:00:05", \ > "10.0.0.1") > > +OVS_WAIT_UNTIL([test x$(ovn-nbctl lsp-get-up sw0-p1-f) = xup]) > +OVS_WAIT_UNTIL([test x$(ovn-nbctl lsp-get-up sw0-p2-f) = xup]) > +OVS_WAIT_UNTIL([test x$(ovn-nbctl lsp-get-up sw0-p3-f) = xup]) > + > # Send ping from sw0-p1-f to sw0-p3-f > NS_CHECK_EXEC([sw0-p1-f], [ping -q -c 3 -i 0.3 -w 2 10.0.0.5 | FORMAT_PING], > \ > [0], [dnl >
Looks good to me, thanks! Acked-by: Dumitru Ceara <[email protected]> _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
