HomeClusterLabs Projects

Refactor: cts: Drop --experimental-tests option from cts-lab

Description

Refactor: cts: Drop --experimental-tests option from cts-lab

SplitBrainTest has been marked as experimental since 2009. There are no
other experimental tests, and this option is False by default. So
SplitBrainTest has been excluded from most of our cts-lab runs.

The experimental flag was set on SplitBrainTest by commit 84d30e0. As of
that commit, the "is_unsafe" flag already existed. "Unsafe" tests are
included by default but can be excluded by the "--no-unsafe-tests"
option. "Experimental" tests are excluded by default but can be included
by the "--experimental-tests" option.

That commit message doesn't give any suggestion of why SplitBrainTest
was marked experimental instead of unsafe. The message says only "A
bunch of cleanups so that we dont peform inappropriate test when ocfs2
is active". That seems like the intended use case for the is_unsafe
flag:

--no-unsafe-tests     Don't run tests that are unsafe for use with
                      ocfs2/drbd

In any case, if a user with ocfs2 was concerned about the safety of
their cluster, they were probably passing --no-unsafe-tests and NOT
passing --experimental-tests. (Supposing for a moment that there were
any external users.) So this should result in no practical changes.

P.S. When the --experimental option itself was added -- NOT when the
is_experimental flag was set for SplitBrainTest -- the --no-unsafe-tests
option and the is_unsafe flag did not yet exist. See 0eb08ff3.

Signed-off-by: Reid Wahl <nrwahl@protonmail.com>

Details

Provenance
nrwahl2Authored on Mon, Aug 18, 2:24 PM
Parents
rPccbf76519e46: Refactor: cts: Drop --facility option from cts-lab
Branches
Unknown
Tags
Unknown