Re: [ovs-dev] [PATCH v2] ovn: Add a new logical switch port type - 'virtual'

2019-05-16 Thread Han Zhou
On Thu, May 16, 2019 at 11:40 AM Numan Siddique wrote: > > > On Thu, May 16, 2019 at 10:20 PM Han Zhou wrote: > >> >> >> On Wed, May 15, 2019 at 11:59 PM Numan Siddique >> wrote: >> >>> >>> >>> On Thu, May 16, 2019 at 1:12 AM Han Zhou wrote: >>> On Wed, May 15, 2019 at 11:55 AM

Re: [ovs-dev] [PATCH v2] ovn: Add a new logical switch port type - 'virtual'

2019-05-16 Thread Numan Siddique
On Thu, May 16, 2019 at 10:20 PM Han Zhou wrote: > > > On Wed, May 15, 2019 at 11:59 PM Numan Siddique > wrote: > >> >> >> On Thu, May 16, 2019 at 1:12 AM Han Zhou wrote: >> >>> >>> >>> On Wed, May 15, 2019 at 11:55 AM Numan Siddique >>> wrote: >>> On Thu, May 16, 2019 at 12:10

Re: [ovs-dev] [PATCH v2] ovn: Add a new logical switch port type - 'virtual'

2019-05-16 Thread Han Zhou
On Wed, May 15, 2019 at 11:59 PM Numan Siddique wrote: > > > On Thu, May 16, 2019 at 1:12 AM Han Zhou wrote: > >> >> >> On Wed, May 15, 2019 at 11:55 AM Numan Siddique >> wrote: >> >>> >>> >>> On Thu, May 16, 2019 at 12:10 AM Han Zhou wrote: >>> On Wed, May 15, 2019 at 11:36 AM

Re: [ovs-dev] [PATCH v2] ovn: Add a new logical switch port type - 'virtual'

2019-05-16 Thread Numan Siddique
On Thu, May 16, 2019 at 1:12 AM Han Zhou wrote: > > > On Wed, May 15, 2019 at 11:55 AM Numan Siddique > wrote: > >> >> >> On Thu, May 16, 2019 at 12:10 AM Han Zhou wrote: >> >>> >>> >>> On Wed, May 15, 2019 at 11:36 AM wrote: >>> > >>> > From: Numan Siddique >>> > >>> > This new type is

Re: [ovs-dev] [PATCH v2] ovn: Add a new logical switch port type - 'virtual'

2019-05-15 Thread Han Zhou
On Wed, May 15, 2019 at 11:55 AM Numan Siddique wrote: > > > On Thu, May 16, 2019 at 12:10 AM Han Zhou wrote: > >> >> >> On Wed, May 15, 2019 at 11:36 AM wrote: >> > >> > From: Numan Siddique >> > >> > This new type is added for the following reasons: >> > >> > - When a load balancer is

Re: [ovs-dev] [PATCH v2] ovn: Add a new logical switch port type - 'virtual'

2019-05-15 Thread 0-day Robot
Bleep bloop. Greetings Numan Siddique, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Line is 190 characters long (recommended limit is 79) #496 FILE:

Re: [ovs-dev] [PATCH v2] ovn: Add a new logical switch port type - 'virtual'

2019-05-15 Thread Numan Siddique
On Thu, May 16, 2019 at 12:10 AM Han Zhou wrote: > > > On Wed, May 15, 2019 at 11:36 AM wrote: > > > > From: Numan Siddique > > > > This new type is added for the following reasons: > > > > - When a load balancer is created in an OpenStack deployment with > Octavia > > service, it

Re: [ovs-dev] [PATCH v2] ovn: Add a new logical switch port type - 'virtual'

2019-05-15 Thread Han Zhou
On Wed, May 15, 2019 at 11:36 AM wrote: > > From: Numan Siddique > > This new type is added for the following reasons: > > - When a load balancer is created in an OpenStack deployment with Octavia > service, it creates a logical port 'VIP' for the virtual ip. > > - This logical port is

[ovs-dev] [PATCH v2] ovn: Add a new logical switch port type - 'virtual'

2019-05-15 Thread nusiddiq
From: Numan Siddique This new type is added for the following reasons: - When a load balancer is created in an OpenStack deployment with Octavia service, it creates a logical port 'VIP' for the virtual ip. - This logical port is not bound to any VIF. - Octavia service creates a