Re: [ovs-dev] [PATCH] ovn.at: A "peer" is only for interconnected routers.

2016-07-19 Thread Ryan Moats
"dev" wrote on 07/19/2016 11:08:10 AM: > From: Ben Pfaff > To: Guru Shetty > Cc: ovs dev > Date: 07/19/2016 11:08 AM > Subject: Re: [ovs-dev] [PATCH] ovn.at: A "peer" is only for > interconnected routers. > Sent by: "dev" > > On Tue,

Re: [ovs-dev] [PATCH] ovn.at: A "peer" is only for interconnected routers.

2016-07-19 Thread Ben Pfaff
On Tue, Jul 19, 2016 at 08:44:23AM -0700, Ben Pfaff wrote: > On Mon, Jul 18, 2016 at 01:57:53PM -0700, Guru Shetty wrote: > I posted these as proper patches in case anyone has a comment: > https://patchwork.ozlabs.org/patch/650263/ > https://patchwork.ozlabs.org/patch/650264/ > > I

Re: [ovs-dev] [PATCH] ovn.at: A "peer" is only for interconnected routers.

2016-07-19 Thread Ben Pfaff
On Mon, Jul 18, 2016 at 01:57:53PM -0700, Guru Shetty wrote: > On 18 July 2016 at 13:11, Ben Pfaff wrote: > > > On Mon, Jul 18, 2016 at 11:34:54AM -0700, Guru Shetty wrote: > > > On 18 July 2016 at 10:35, Ben Pfaff wrote: > > > > > > > On Mon, Jul 18, 2016 at 12:00:30AM -0700, Gurucharan Shetty

Re: [ovs-dev] [PATCH] ovn.at: A "peer" is only for interconnected routers.

2016-07-18 Thread Guru Shetty
On 18 July 2016 at 13:11, Ben Pfaff wrote: > On Mon, Jul 18, 2016 at 11:34:54AM -0700, Guru Shetty wrote: > > On 18 July 2016 at 10:35, Ben Pfaff wrote: > > > > > On Mon, Jul 18, 2016 at 12:00:30AM -0700, Gurucharan Shetty wrote: > > > > We should not use "peer" while connecting a router to a sw

Re: [ovs-dev] [PATCH] ovn.at: A "peer" is only for interconnected routers.

2016-07-18 Thread Ben Pfaff
On Mon, Jul 18, 2016 at 11:34:54AM -0700, Guru Shetty wrote: > On 18 July 2016 at 10:35, Ben Pfaff wrote: > > > On Mon, Jul 18, 2016 at 12:00:30AM -0700, Gurucharan Shetty wrote: > > > We should not use "peer" while connecting a router to a switch. > > > (Doing so, will cause ovn-northd to consta

Re: [ovs-dev] [PATCH] ovn.at: A "peer" is only for interconnected routers.

2016-07-18 Thread Guru Shetty
On 18 July 2016 at 10:35, Ben Pfaff wrote: > On Mon, Jul 18, 2016 at 12:00:30AM -0700, Gurucharan Shetty wrote: > > We should not use "peer" while connecting a router to a switch. > > (Doing so, will cause ovn-northd to constantly create and destroy > > port_binding records which causes CPU utili

Re: [ovs-dev] [PATCH] ovn.at: A "peer" is only for interconnected routers.

2016-07-18 Thread Flaviof
On Mon, Jul 18, 2016 at 3:00 AM, Gurucharan Shetty wrote: > We should not use "peer" while connecting a router to a switch. > (Doing so, will cause ovn-northd to constantly create and destroy > port_binding records which causes CPU utilization of ovn-controller to > spike up.) > > Fixes: 31114af7

Re: [ovs-dev] [PATCH] ovn.at: A "peer" is only for interconnected routers.

2016-07-18 Thread Ben Pfaff
On Mon, Jul 18, 2016 at 12:00:30AM -0700, Gurucharan Shetty wrote: > We should not use "peer" while connecting a router to a switch. > (Doing so, will cause ovn-northd to constantly create and destroy > port_binding records which causes CPU utilization of ovn-controller to > spike up.) > > Fixes:

[ovs-dev] [PATCH] ovn.at: A "peer" is only for interconnected routers.

2016-07-18 Thread Gurucharan Shetty
We should not use "peer" while connecting a router to a switch. (Doing so, will cause ovn-northd to constantly create and destroy port_binding records which causes CPU utilization of ovn-controller to spike up.) Fixes: 31114af758c7e6 ("ovn-nbctl: Update logical router port commands.") Signed-off-b