Re: [ovs-dev] [PATCH] OVN localport type support

2017-05-10 Thread Daniel Alvarez Sanchez
On Tue, May 9, 2017 at 9:17 PM, Ben Pfaff wrote: > Thanks. > > This is a good summary of some of what the patch does. Should it be in > the commit message? > ack I ommited the commands and outputs in the commit message. In order to illustrate the purpose of this patch: - One

Re: [ovs-dev] [PATCH] OVN localport type support

2017-05-09 Thread Ben Pfaff
Thanks. This is a good summary of some of what the patch does. Should it be in the commit message? On Tue, May 09, 2017 at 05:09:43PM +0200, Daniel Alvarez Sanchez wrote: > Hi, > > I've submitted a new patch v3 where I removed the external-id > "ovn-localport" > from the Interface which I used

Re: [ovs-dev] [PATCH] OVN localport type support

2017-05-09 Thread Daniel Alvarez Sanchez
Hi, I've submitted a new patch v3 where I removed the external-id "ovn-localport" from the Interface which I used to identify a port as localport in physical.c. Instead, I have passed another parameter "local_lports" to physical_run. When inserting flows in table 32, I'm inserting higher

Re: [ovs-dev] [PATCH] OVN localport type support

2017-05-05 Thread Ben Pfaff
[oops, adding back the list] On Fri, May 05, 2017 at 08:51:01AM -0700, Ben Pfaff wrote: > On Fri, May 05, 2017 at 02:58:45PM +0200, Daniel Alvarez Sanchez wrote: > > Thanks a lot Ben for taking the time to review the patch and submit > > the 3 patch series. > > > > On Wed, May 3, 2017 at 11:54

Re: [ovs-dev] [PATCH] OVN localport type support

2017-05-02 Thread Ben Pfaff
On Tue, May 02, 2017 at 03:57:27PM -0400, Russell Bryant wrote: > On Tue, May 2, 2017 at 11:31 AM, Ben Pfaff wrote: > > On Tue, Apr 25, 2017 at 11:05:28AM +, Daniel Alvarez wrote: > >> This patch introduces a new type of OVN ports called "localport". > >> These ports will be

Re: [ovs-dev] [PATCH] OVN localport type support

2017-05-02 Thread Russell Bryant
On Tue, May 2, 2017 at 11:31 AM, Ben Pfaff wrote: > On Tue, Apr 25, 2017 at 11:05:28AM +, Daniel Alvarez wrote: >> This patch introduces a new type of OVN ports called "localport". >> These ports will be present in every hypervisor and may have the >> same IP/MAC addresses. They

Re: [ovs-dev] [PATCH] OVN localport type support

2017-05-02 Thread Ben Pfaff
On Tue, Apr 25, 2017 at 11:05:28AM +, Daniel Alvarez wrote: > This patch introduces a new type of OVN ports called "localport". > These ports will be present in every hypervisor and may have the > same IP/MAC addresses. They are not bound to any chassis and traffic > to these ports will never