[ovs-dev] [PATCH] ovn-controller-vtep: occasional failure in "binding 1" test case

[Lance Richardson]

> > diff --git a/tests/ovn-controller-vtep.at b/tests/ovn-controller-vtep.at
> > index c296f0e..c978137 100644
> > --- a/tests/ovn-controller-vtep.at
> > +++ b/tests/ovn-controller-vtep.at
> > @@ -225,8 +225,9 @@ AT_CHECK([sed -n 's/^.*\(|WARN|.*\)$/\1/p'
> > ovn-controller-vtep.log | sed 's/([[-
> > 
> >  # deletes physical ports from vtep.
> >  AT_CHECK([ovs-vsctl del-port p0 -- del-port p1])
> > -AT_CHECK([vtep-ctl del-port br-vtep p0 -- del-port br-vtep p1])
> >  OVS_WAIT_UNTIL([test -z "`ovn-sbctl list Chassis | grep --
> > br-vtep_lswitch`"])
> > +OVS_WAIT_UNTIL([test -z "`vtep-ctl list physical_port p0`"])
> > +OVS_WAIT_UNTIL([test -z "`vtep-ctl list physical_port p1`"])
> >  # should see empty chassis column in both binding entries.
> >  AT_CHECK_UNQUOTED([ovn-sbctl --columns=chassis list Port_Binding | cut -d
> > ':' -f2 | tr -d ' ' | sort], [0], [dnl
> > 
> > 
> >  What do you think ?
> >
> 
> Hi Darrell,
> 
> That's a good idea, I'll give it a try.
> 
> Thanks,
> 
>    Lance
> > 
> > 
> > > --
> > > 2.5.5
> > >
> > > _______________________________________________
> > > dev mailing list
> > > dev at openvswitch.org
> > > http://openvswitch.org/mailman/listinfo/dev
> > >
> > 
> 

Hi Darrell,

This looks good in my testing.  Would you like me to post a v2 with
a Suggested-by:, or were you going to post your version formally?

Thanks,

  Lance