On Wed, Jul 13, 2016 at 2:15 PM, Lance Richardson <lrich...@redhat.com>
wrote:

> > > 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@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
>


please go ahead and post V2

Thanks Darrell
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to