[ovs-dev] [PATCH] ovn-controller: Combine two Logical_Flow loops into one.

[Russell Bryant]

On Thu, Aug 11, 2016 at 8:09 AM, nickcooper-zhangtonghao <
nickcooper-zhangtonghao at opencloud.tech> wrote:

>
> > On Aug 7, 2016, at 6:54 AM, Ryan Moats <rmoats at us.ibm.com> wrote:
> >
> > > From: nickcooper-zhangtonghao <nickcooper-zhangtonghao at opencloud.tech
> <mailto:nickcooper-zhangtonghao at opencloud.tech>>
> > > To: dev at openvswitch.org <mailto:dev at openvswitch.org>
> > > Cc: nickcooper-zhangtonghao <nickcooper-zhangtonghao at opencloud.tech
> <mailto:nickcooper-zhangtonghao at opencloud.tech>>
> > > Date: 08/06/2016 06:43 AM
> > > Subject: [ovs-dev] [PATCH] ovn-controller: Combine two Logical_Flow
> > > loops into one.
> > > Sent by: "dev" <dev-bounces at openvswitch.org <mailto:dev-bounces@
> openvswitch.org>>
> > >
> > > Signed-off-by: nickcooper-zhangtonghao <nickcooper-
> > > zhangtonghao at opencloud.tech>
> > > ---
> > >  ovn/controller/lflow.c | 14 +++++---------
> > >  1 file changed, 5 insertions(+), 9 deletions(-)
>


> > I had to go back through the July archives to remember why we ended
> > up with two logical flow loops and this change was always on my
> > radar, but I just forgot about it while chasing other higher priority
> > items.  So, I went ahead and ran through the ovn unit tests with
> > -j4 50 times to verify that the test failure rate I'm seeing is
> > reasonable (I didn't get one...), so
> >
> > Acked-by: Ryan Moats <rmoats at us.ibm.com <mailto:rmoats at us.ibm.com>>
>

Thanks, I applied this to master.

-- 
Russell Bryant