"dev" <dev-boun...@openvswitch.org> wrote on 07/18/2016 01:34:02 PM:
> From: Ben Pfaff <b...@ovn.org> > To: Lance Richardson <lrich...@redhat.com> > Cc: dev@openvswitch.org > Date: 07/18/2016 01:34 PM > Subject: Re: [ovs-dev] [v1 1/1] Fix "raceful" E2E ovn tests > Sent by: "dev" <dev-boun...@openvswitch.org> > > On Mon, Jul 18, 2016 at 01:50:40PM -0400, Lance Richardson wrote: > > ----- Original Message ----- > > > From: "Ben Pfaff" <b...@ovn.org> > > > To: "Amitabha Biswas" <abis...@us.ibm.com> > > > Cc: dev@openvswitch.org > > > Sent: Monday, July 18, 2016 1:33:56 PM > > > Subject: Re: [ovs-dev] [v1 1/1] Fix "raceful" E2E ovn tests > > > > > > On Fri, May 20, 2016 at 11:17:57AM -0700, Amitabha Biswas wrote: > > > > This patch fixes the "raceful" situation that occurs when ovn-nbctl > > > > configures the OVN Northbound database till the point when the > > > > ovn-controller picks up the corresponding logical flows and applies > > > > them as OpenFlow rules on the hypervisor. > > > > > > I posted a patch that does something similar: > > > http://openvswitch.org/pipermail/dev/2016-July/075516.html > > > > > > I forgot, when I started writing it, that this previous patch had > > > existed. > > > > I have spent some time recently looking into the root causes of OVN > > E2E test failures. Initially I was thinking that this would lead to > > a better understanding of these races, but as it turned out it seems > > that most of these failures are not really due to races at all, but > > are instead mostly due to other types of bugs. Here's the list: > > > > > > https://patchwork.ozlabs.org/patch/631074/ (merged) > > https://patchwork.ozlabs.org/patch/643536/ (merged) > > https://patchwork.ozlabs.org/patch/645650/ (queued for review) > > https://patchwork.ozlabs.org/patch/646179/ (queued for review) > > https://patchwork.ozlabs.org/patch/646257/ (queued for review) > > https://patchwork.ozlabs.org/patch/648815/ (merged) > > Hmm, that's interesting. I guess that we are all too eager to blame > bugs on "races", when in fact there can be (and it appears that there > are in practice!) many root causes for bugs. I guess that fixing the > races, then, will make it more likely that we won't ignore other bugs. True. I always viewed the concept behind your CMS patch as being more around the operational improvement it brings to the table (which your patch calls out). Avoiding race conditions was a nice win on the side. Ryan _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
