[ovs-dev] [PATCH] ovn-controller: Assign ct_zone id to local datapaths instead of lports

2017-02-23 Thread nusiddiq
From: Numan Siddique Having zone id per datapath is more than sufficient, because the CT tuple information will be unique anyway with in the logical datapath. In our testing we have observed that, the packet between two ports of a datapath within the same chassis is sent to the CT twice (both in

Re: [ovs-dev] [PATCH] ovn-controller: Assign ct_zone id to local datapaths instead of lports

2017-02-23 Thread Russell Bryant
On Thu, Feb 23, 2017 at 9:04 AM, wrote: > From: Numan Siddique > > Having zone id per datapath is more than sufficient, because the > CT tuple information will be unique anyway with in the logical > datapath. > > In our testing we have observed that, the packet between two ports of > a datapath

Re: [ovs-dev] [PATCH] ovn-controller: Assign ct_zone id to local datapaths instead of lports

2017-02-23 Thread Mickey Spiegel
On Thu, Feb 23, 2017 at 6:04 AM, wrote: > From: Numan Siddique > > Having zone id per datapath is more than sufficient, because the > CT tuple information will be unique anyway with in the logical > datapath. > This proposal conflicts with another proposal that is currently in flight ( https://

Re: [ovs-dev] [PATCH] ovn-controller: Assign ct_zone id to local datapaths instead of lports

2017-02-23 Thread Numan Siddique
​​ On Thu, Feb 23, 2017 at 11:06 PM, Mickey Spiegel wrote: > > On Thu, Feb 23, 2017 at 6:04 AM, wrote: > >> From: Numan Siddique >> >> Having zone id per datapath is more than sufficient, because the >> CT tuple information will be unique anyway with in the logical >> datapath. >> > > This pr

Re: [ovs-dev] [PATCH] ovn-controller: Assign ct_zone id to local datapaths instead of lports

2017-02-27 Thread Russell Bryant
On Fri, Feb 24, 2017 at 1:10 AM, Numan Siddique wrote: > ​​ > > > On Thu, Feb 23, 2017 at 11:06 PM, Mickey Spiegel > wrote: > > > > > On Thu, Feb 23, 2017 at 6:04 AM, wrote: > > > >> From: Numan Siddique > >> > >> Having zone id per datapath is more than sufficient, because the > >> CT tuple i