Re: [ovs-dev] [PATCH v4] ovn: Support a new Logical_Switch_Port.type - 'external'

2019-01-17 Thread Numan Siddique
On Thu, Jan 17, 2019 at 10:57 PM Han Zhou wrote: > On Tue, Jan 15, 2019 at 10:24 AM Numan Siddique > wrote: > > > > > > > > On Tue, Jan 15, 2019 at 5:42 AM Han Zhou wrote: > >> > >> Hi Numan, > >> > >> The feature looks very good overall. I have some more comments inlined. > > > > > > Thank

Re: [ovs-dev] [PATCH v4] ovn: Support a new Logical_Switch_Port.type - 'external'

2019-01-17 Thread Han Zhou
On Tue, Jan 15, 2019 at 10:24 AM Numan Siddique wrote: > > > > On Tue, Jan 15, 2019 at 5:42 AM Han Zhou wrote: >> >> Hi Numan, >> >> The feature looks very good overall. I have some more comments inlined. > > > Thank Han for the review and the comments. > > You are right. The logical port's mac

Re: [ovs-dev] [PATCH v4] ovn: Support a new Logical_Switch_Port.type - 'external'

2019-01-15 Thread Numan Siddique
On Tue, Jan 15, 2019 at 5:42 AM Han Zhou wrote: > Hi Numan, > > The feature looks very good overall. I have some more comments inlined. > Thank Han for the review and the comments. You are right. The logical port's mac flaps too. I had to read the commit message again to recall :) > On Thu,

Re: [ovs-dev] [PATCH v4] ovn: Support a new Logical_Switch_Port.type - 'external'

2019-01-14 Thread Han Zhou
Hi Numan, The feature looks very good overall. I have some more comments inlined. On Thu, Jan 10, 2019 at 12:37 PM wrote: > > From: Numan Siddique > > In the case of OpenStack + OVN, when the VMs are booted on > hypervisors supporting SR-IOV nics, there are no OVS ports > for these VMs. When

[ovs-dev] [PATCH v4] ovn: Support a new Logical_Switch_Port.type - 'external'

2019-01-10 Thread nusiddiq
From: Numan Siddique In the case of OpenStack + OVN, when the VMs are booted on hypervisors supporting SR-IOV nics, there are no OVS ports for these VMs. When these VMs sends DHCPv4, DHPCv6 or IPv6 Router Solicitation requests, the local ovn-controller cannot reply to these packets. OpenStack