Re: [ovs-dev] [PATCH] ovn-controller: Drop remove_local_datapath_by_binding().

2016-07-18 Thread Ben Pfaff
On Mon, Jul 18, 2016 at 03:56:58PM -0500, Ryan Moats wrote: > P.S. Ben, I'll be sending a rebased pair of incremental processing > patches later this evening, after doubles league. I'm getting back into "review mode" right now, so it's good timing. With luck, I'll just apply them quickly this time

Re: [ovs-dev] [PATCH] ovn-controller: Drop remove_local_datapath_by_binding().

2016-07-18 Thread Russell Bryant
On Mon, Jul 18, 2016 at 4:56 PM, Ryan Moats wrote: > Russell Bryant wrote on 07/18/2016 03:47:34 PM: > > > From: Russell Bryant > > To: dev@openvswitch.org > > Cc: Ryan Moats/Omaha/IBM@IBMUS, Russell Bryant > > Date: 07/18/2016 03:47 PM > > Subject: [PATCH] ovn-controller: Drop remove_local_da

Re: [ovs-dev] [PATCH] ovn-controller: Drop remove_local_datapath_by_binding().

2016-07-18 Thread Ryan Moats
Russell Bryant wrote on 07/18/2016 03:47:34 PM: > From: Russell Bryant > To: dev@openvswitch.org > Cc: Ryan Moats/Omaha/IBM@IBMUS, Russell Bryant > Date: 07/18/2016 03:47 PM > Subject: [PATCH] ovn-controller: Drop remove_local_datapath_by_binding(). > > ovn-controller has an hmap called 'local_

[ovs-dev] [PATCH] ovn-controller: Drop remove_local_datapath_by_binding().

2016-07-18 Thread Russell Bryant
ovn-controller has an hmap called 'local_datapaths' which tracks all OVN datapaths that have at least one port binding on the local chassis. This patch corrects the case where a port binding row is deleted from the southbound DB while it's still bound to the chassis, meaning it was deleted before