On Thu, Sep 24, 2015 at 01:27:07PM -0700, Justin Pettit wrote:
> 
> > On Sep 11, 2015, at 8:53 PM, Ben Pfaff <b...@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 <jpet...@nicira.com>

Thanks, I'm going to run this again and then push it.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to