[ovs-dev] [PATCH] ovn-controller: squelch expected duplicate flow warnings

[Ben Pfaff]

On Tue, Jul 26, 2016 at 01:54:29PM -0700, Guru Shetty wrote:
> On 24 July 2016 at 10:07, Ryan Moats <rmoats at us.ibm.com> wrote:
> 
> > In the physical processing of ovn-controller, there are two
> > sets of OF flows that are still fully recalculated every cycle:
> >
> >   Flows that aren't associated with any logical flow, and
> >   Flows calculated based on multicast groups
> >
> > Because these flows are recalculated fully each cycle, full
> > duplicates of existing OF flows are created and the OF management
> > code in ovn-controller pollutes the logs with false positive
> > warnings about repeated duplicates.
> >
> > As a short term measure, ignore full duplicates for both of
> > these types of flows, but still warn if the action changes
> > (as that is not expected and may be indicative of a problem).
> >
> > Signed-off-by: Ryan Moats <rmoats at us.ibm.com>
> >
> 
> I also noticed that "commit 70c7cfef188b5ae9940abd5 (ovn-controller: Add
> incremental processing to lflow_run and physical_run)" causes load
> balancing system unit tests to fail. A little debugging shows that groups
> are getting deleted when new flows are added.  My hunch is that this is
> likely because 'desired_groups' in ofctl_put gets deleted in every run. But
> in the next run, it does not get updated as we no longer process all flows.

It's unclear to me from the discussion of this patch whether it's
beneficial.  Can someone clarify?

Thanks,

Ben.