Re: [ovs-dev] [PATCH] ovn-controller: Fix table 33 flows for containers.

2015-09-28 Thread Gurucharan Shetty
I will send a v2. On Mon, Sep 28, 2015 at 12:28 PM, Gurucharan Shetty wrote: >> From first read of the patch, it looks like this doesn't change >> anything. The above code still seems to provide the same result. The >> important part is actually in code not changed. There is code later >> that

Re: [ovs-dev] [PATCH] ovn-controller: Fix table 33 flows for containers.

2015-09-28 Thread Gurucharan Shetty
> From first read of the patch, it looks like this doesn't change > anything. The above code still seems to provide the same result. The > important part is actually in code not changed. There is code later > that uses whether or not the port is in lport_to_ofport to determine if > it's local o

Re: [ovs-dev] [PATCH] ovn-controller: Fix table 33 flows for containers.

2015-09-28 Thread Russell Bryant
On 09/28/2015 09:54 AM, Gurucharan Shetty wrote: > When containers are inside a VM, the broadcast flow > generated in table 33 was faulty. This was because > 'lport_to_ofport' did not include logical ports > associated with containers. > > Signed-off-by: Gurucharan Shetty > --- > ovn/controller/

[ovs-dev] [PATCH] ovn-controller: Fix table 33 flows for containers.

2015-09-28 Thread Gurucharan Shetty
When containers are inside a VM, the broadcast flow generated in table 33 was faulty. This was because 'lport_to_ofport' did not include logical ports associated with containers. Signed-off-by: Gurucharan Shetty --- ovn/controller/physical.c | 27 --- 1 file changed, 20 i