Ben Pfaff <b...@ovn.org> wrote on 06/24/2016 02:59:30 PM:

> From: Ben Pfaff <b...@ovn.org>
> To: Russell Bryant <russ...@ovn.org>
> Cc: dev@openvswitch.org, Ryan Moats/Omaha/IBM@IBMUS
> Date: 06/24/2016 03:00 PM
> Subject: Re: [PATCH 0/4] Restore ovn-controller binding functionality.
>
> On Fri, Jun 24, 2016 at 03:52:07PM -0400, Russell Bryant wrote:
> > "Convert binding_run to incremental processing" caused a regression
> > in port binding handling.  This was seen in OpenStack CI.  The last
> > commit here adds a test case that would have caught the issue in OVN.
> >
> > There are 3 reverts as two later patches conflicted with reverting
> > patch 3/4.  This is just to show what has to be rolled back to get
> > to where the added test in patch 4/4 will pass.
> >
> > Ryan says he has a possible fix for the issue, so we should pursue
> > that before merging the reverts.
>
> This is interesting.  I expected some minor fallout, but I did not
> expect it to break severely.  Incremental processing is hard.
>
> Ryan, will your patch be ready soon?  At a glance, I do not see it yet.
>

It is hard, but I have a patch that looks hopeful - I've tested it
by hand, I'm running it through existing UT first and then I'll add
Russell's test and run it through UT again and then post it. I'd say
ETA of 30 minutes or less...

Ryan
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to