Ben Pfaff <[email protected]> wrote on 07/27/2016 03:53:56 PM:
> From: Ben Pfaff <[email protected]> > To: Guru Shetty <[email protected]> > Cc: Ryan Moats/Omaha/IBM@IBMUS, ovs dev <[email protected]> > Date: 07/27/2016 03:54 PM > Subject: Re: [ovs-dev] [PATCH] ovn-controller: squelch expected > duplicate flow warnings > > On Tue, Jul 26, 2016 at 01:54:29PM -0700, Guru Shetty wrote: > > On 24 July 2016 at 10:07, Ryan Moats <[email protected]> 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 <[email protected]> > > > > > > > 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. Unfortunately, I don't think we maintained reasonable email thread discipline on this discussion - for me, the vast majority of the conversation was about the CPU cycle bug that Guru mentions and not the patch itself. The conditions stated in the commit message still apply even after fixing Guru's issue and so I still see the squelching as being useful to avoid polluting the ovn-controller logs with false positive messages... Ryan _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
