[ovs-dev] [PATCH] tests: Fix occasional failure of "self-linking idl, inconsistent ops" test.

Ben Pfaff blp at nicira.com
Wed Jun 23 16:48:29 UTC 2010


On Tue, Jun 22, 2010 at 06:10:50PM -0700, Jesse Gross wrote:
> On Tue, Jun 22, 2010 at 2:48 PM, Ben Pfaff <blp at nicira.com> wrote:
> 
> > This test is nondeterministic: depending on the ordering of the random
> > row UUIDs, the error message might mention one of two different nonexistent
> > rows that are referenced.  This commit fixes the problem by accepting
> > either one of the two correct rows.
> 
> If it works consistently, it looks fine.  It seems like this problem
> started happening fairly recently.  Do you know what triggered it?

It was this commit:

    commit 97f7803b8eb6f9967ed6090eec1cd2533bad2afe
    Author: Ben Pfaff <blp at nicira.com>
    Date:   Fri Jun 11 13:56:36 2010 -0700

        ovsdb-server: Improve logging for referential integrity violations.

        This may help with bug #2727 "ovs-vsctl transaction error while changing
        VM power state", if it is reproducible.

Apparently the author only ran the testsuite once before he pushed it
out.  It's that "blp at nicira.com" guy again.  He writes more buggy tests
than anyone else around here.




More information about the dev mailing list