[Gluster-devel] IMPORTANT - Adding further volume types to our smoke tests

Jeff Darcy jdarcy at redhat.com
Wed Nov 12 22:21:31 UTC 2014


\> At the moment, our smoke tests in Jenkins only run on a
> replicated volume.  Extending that out to other volume types
> should (in theory :>) help catch other simple gotchas.
> 
> Xavi has put together a patch for doing just this, which I'd
> like to apply and get us running:
> 
>   https://forge.gluster.org/gluster-patch-acceptance-tests/gluster-patch-acceptance-tests/merge_requests/4
> 
> What are people's thoughts on the general idea, and on the
> above proposed patch?  (The Forge isn't using Gerrit, so
> review/comments back here please :>)

I'm ambivalent.  On the one hand, I think this is an important
step in the right direction.  Sometimes we need to be able to
run *all* of our existing tests with some feature enabled, not
just a few feature-specific tests.  SSL is an example of this,
and transport (or other forms of) multi-threading will be as
well.

On the other hand, I'm not sure smoke is the place to do this.
Smoke is supposed to be a *quick* test to catch *basic* errors
(e.g. source fails to build) before we devote hours to a full
regression test.  How much does this change throughput on the
smoke-test queue?  Should we be doing this in regression
instead, or in a third testing tier between the two we have?

My gut feel is that we need to think more about how to run
a matrix of M tests across N configurations, instead of just
putting feature/regression tests and configuration tests into
one big bucket.  Or maybe that's a longer-term thing.


More information about the Gluster-devel mailing list