[ovs-dev] [PATCH] ovn: Implement basic end-to-end full mesh test.

Ben Pfaff blp at nicira.com
Tue Sep 29 22:41:17 UTC 2015


On Thu, Sep 24, 2015 at 01:27:07PM -0700, Justin Pettit wrote:
> 
> > On Sep 11, 2015, at 8:53 PM, Ben Pfaff <blp at nicira.com> wrote:
> > 
> > +    mac=`ovs-vsctl get Interface $bridge mac_in_use | sed s/\"//g`
> > +    arp_table="$arp_table $sandbox,$bridge,$ip,$mac"
> > +    ovs-appctl netdev-dummy/ip4addr $bridge $ip/$masklen >/dev/null || return 1
> > +    ovs-appctl ovs/route/add $ip/$masklen $bridge >/dev/null || return 1
> > +    ovs-vsctl \
> > +	-- set Open_vSwitch . external-ids:system-id=$sandbox \
> > +        -- set Open_vSwitch . external-ids:ovn-remote=unix:$ovs_base/ovn-sb/ovn-sb.sock \
> 
> Super minor nit, but it looks like there are two extra spaces here.

It looks like spaces and tabs got mixed together in this file.  I've now
converted all of them to spaces.

> This is very cool and much smaller than I expected.  Thanks!
> 
> Acked-by: Justin Pettit <jpettit at nicira.com>

Thanks, I'm going to run this again and then push it.



More information about the dev mailing list