On Tue, Apr 08, 2014 at 09:39:57AM +0900, YAMAMOTO Takashi wrote: > > This corrects an inconsistency in the MPLS tests and > > avoids a race by waiting for sufficient output before > > checking its contents > > this one looks identical to: http://patchwork.openvswitch.org/patch/3579/
Agreed. > > YAMAMOTO Takashi > > > > > Signed-off-by: Simon Horman <ho...@verge.net.au> > > --- > > tests/ofproto-dpif.at | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/tests/ofproto-dpif.at b/tests/ofproto-dpif.at > > index f2ddda9..1749f60 100644 > > --- a/tests/ofproto-dpif.at > > +++ b/tests/ofproto-dpif.at > > @@ -988,8 +988,9 @@ dnl > > in_port(1),eth(src=00:01:02:03:04:05,dst=10:11:12:13:14:15),eth_type(0x8847) > > for i in 1 2 3; do > > ovs-appctl netdev-dummy/receive p1 > > 'in_port(1),eth(src=40:44:44:44:44:43,dst=50:54:00:00:00:07),eth_type(0x8847),mpls(label=11,tc=3,ttl=64,bos=1)' > > done > > - > > +OVS_WAIT_UNTIL([test `wc -l < ofctl_monitor.log` -ge 6]) > > OVS_APP_EXIT_AND_WAIT(ovs-ofctl) > > + > > AT_CHECK([cat ofctl_monitor.log], [0], [dnl > > NXT_PACKET_IN (xid=0x0): cookie=0xa total_len=64 in_port=1 (via action) > > data_len=64 (unbuffered) > > > > mpls,metadata=0,in_port=0,vlan_tci=0x0000,dl_src=40:44:44:44:44:43,dl_dst=50:54:00:00:00:07,mpls_label=10,mpls_tc=3,mpls_ttl=64,mpls_bos=0,mpls_lse1=46912 > > -- > > 1.8.5.2 > > > > _______________________________________________ > > dev mailing list > > dev@openvswitch.org > > http://openvswitch.org/mailman/listinfo/dev > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
