[ovs-dev] [PATCH] tests: Fix double-rebuild of testsuite for "check-valgrind".

Ben Pfaff blp at ovn.org
Wed Oct 12 19:20:46 UTC 2016


On Wed, Oct 12, 2016 at 10:42:36AM -0700, Ben Pfaff wrote:
> On Tue, Oct 11, 2016 at 04:58:26PM -0700, Andy Zhou wrote:
> > On Wed, Oct 5, 2016 at 6:28 PM, Ben Pfaff <blp at ovn.org> wrote:
> > 
> > > When I ran "make check-valgrind -j10" and the testsuite needed to be
> > > rebuilt, two copies of it were rebuilt in parallel and sometimes they
> > > raced with each other.  I don't have the full story on exactly why this
> > > happened, but this commit, which eliminates redundant dependencies from
> > > check-* targets, fixes the problem for me.  The dependencies are redundant
> > > because these targets depend on "all", which also depends on them.
> > >
> > > Signed-off-by: Ben Pfaff <blp at ovn.org>
> > >
> > 
> > I was not clear to me either why the race happened, but the change looks
> > good to me.
> > 
> > Acked-by: Andy Zhou <azhou at ovn.org>
> > 
> > There are two additional targets that have the similar pattern, wonder if
> > we should change them as well...
> 
> You're right.  I made those changes too.
> 
> My confidence is somewhat shaky on this, but it's easy enough to revert
> if we missed something, so I applied this to master.

Well, at least travis is happy:
        https://travis-ci.org/openvswitch/ovs/builds/167124458
So any bugs are more subtle.



More information about the dev mailing list