[ovs-dev] [PATCH] ovn-controller: update_ct_zone operates always on empty set

2016-07-26 Thread bschanmu
From: Babu Shanmugam Commit 263064a (Convert binding_run to incremental processing.) removed the usage of all_lports from binding_run, but it is infact used in the context of the caller, especially by update_ct_zones(). Without this change, update_ct_zones operates on an empty set always. Sig

Re: [ovs-dev] [PATCH] ovn-controller: update_ct_zone operates always on empty set

2016-07-26 Thread Russell Bryant
On Tue, Jul 26, 2016 at 6:46 AM, wrote: > From: Babu Shanmugam > > Commit 263064a (Convert binding_run to incremental processing.) removed > the usage > of all_lports from binding_run, but it is infact used in the context of > the caller, > especially by update_ct_zones(). > > Without this chang

Re: [ovs-dev] [PATCH] ovn-controller: update_ct_zone operates always on empty set

2016-07-26 Thread Ryan Moats
"dev" wrote on 07/26/2016 08:13:00 PM: > From: Russell Bryant > To: Babu Shanmugam > Cc: ovs dev > Date: 07/26/2016 08:13 PM > Subject: Re: [ovs-dev] [PATCH] ovn-controller: update_ct_zone > operates always on empty set > Sent by: "dev" > > On T

Re: [ovs-dev] [PATCH] ovn-controller: update_ct_zone operates always on empty set

2016-07-26 Thread Guru Shetty
> On Jul 26, 2016, at 6:13 PM, Russell Bryant wrote: > >> On Tue, Jul 26, 2016 at 6:46 AM, wrote: >> >> From: Babu Shanmugam >> >> Commit 263064a (Convert binding_run to incremental processing.) removed >> the usage >> of all_lports from binding_run, but it is infact used in the context of

Re: [ovs-dev] [PATCH] ovn-controller: update_ct_zone operates always on empty set

2016-07-26 Thread Babu Shanmugam
On Wednesday 27 July 2016 06:43 AM, Russell Bryant wrote: On Tue, Jul 26, 2016 at 6:46 AM, > wrote: From: Babu Shanmugam mailto:bscha...@redhat.com>> Commit 263064a (Convert binding_run to incremental processing.) removed the usage of all_lports fro

Re: [ovs-dev] [PATCH] ovn-controller: update_ct_zone operates always on empty set

2016-07-27 Thread Ryan Moats
"dev" wrote on 07/27/2016 12:44:38 AM: > From: Babu Shanmugam > To: Russell Bryant > Cc: ovs dev > Date: 07/27/2016 12:45 AM > Subject: Re: [ovs-dev] [PATCH] ovn-controller: update_ct_zone > operates always on empty set > Sent by: "dev" > >

Re: [ovs-dev] [PATCH] ovn-controller: update_ct_zone operates always on empty set

2016-07-27 Thread Russell Bryant
On Wed, Jul 27, 2016 at 8:34 AM, Ryan Moats wrote: > "dev" wrote on 07/27/2016 12:44:38 AM: > > > From: Babu Shanmugam > > To: Russell Bryant > > Cc: ovs dev > > Date: 07/27/2016 12:45 AM > > Subject: Re: [ovs-dev] [PATCH] ovn-controller: u

Re: [ovs-dev] [PATCH] ovn-controller: update_ct_zone operates always on empty set

2016-07-27 Thread Russell Bryant
On Wed, Jul 27, 2016 at 1:44 AM, Babu Shanmugam wrote: > > > On Wednesday 27 July 2016 06:43 AM, Russell Bryant wrote: > > On Tue, Jul 26, 2016 at 6:46 AM, wrote: > >> From: Babu Shanmugam >> >> Commit 263064a (Convert binding_run to incremental processing.) removed >> the usage >> of all_lport