From a25f4e22f8cc68d95468f6873f4f6f0296d4405e Mon Sep 17 00:00:00 2001 From: Jeff Darcy Date: Sun, 28 Feb 2016 16:34:50 +0530 Subject: cli/tests: disable tiering on NetBSD As proposed in the following email: http://www.gluster.org/pipermail/gluster-devel/2016-February/048464.html Short version: tiering tests on NetBSD are failing often enough to impede other development, and also to cast doubt on whether tiering even works on that platform. The most effective way to satisfy both developers' and users' interests is to disable tiering on NetBSD until such time as someone becomes available to resolve these problems another way. Change-Id: I7375527c339e2650a7e8e900a0d0572013213544 Signed-off-by: Jeff Darcy Reviewed-on: http://review.gluster.org/13535 Smoke: Gluster Build System Reviewed-by: Joseph Fernandes CentOS-regression: Gluster Build System NetBSD-regression: NetBSD Build System --- cli/src/cli-cmd-volume.c | 2 ++ run-tests.sh | 25 +++++++++++++++++++++---- 2 files changed, 23 insertions(+), 4 deletions(-) diff --git a/cli/src/cli-cmd-volume.c b/cli/src/cli-cmd-volume.c index 71221055d58..74f58eeab90 100644 --- a/cli/src/cli-cmd-volume.c +++ b/cli/src/cli-cmd-volume.c @@ -2519,6 +2519,7 @@ struct cli_cmd volume_cmds[] = { cli_cmd_volume_rename_cbk, "rename volume to "},*/ +#if !defined(__NetBSD__) { "volume tier status\n" "volume tier start [force]\n" "volume tier attach [] ...\n" @@ -2538,6 +2539,7 @@ struct cli_cmd volume_cmds[] = { "NOTE: this is old syntax, will be depreciated in next release. " "Please use gluster volume tier detach " "{start|stop|commit} [force]"}, +#endif { "volume add-brick [ ] ... [force]", cli_cmd_volume_add_brick_cbk, diff --git a/run-tests.sh b/run-tests.sh index fe58ed68b56..258beee2b0a 100755 --- a/run-tests.sh +++ b/run-tests.sh @@ -8,6 +8,8 @@ retry="no" tests="" exit_on_failure="yes" +OSTYPE=$(uname -s) + function check_dependencies() { ## Check all dependencies are present @@ -166,7 +168,7 @@ function is_bad_test () ./tests/basic/tier/bug-1214222-directories_missing_after_attach_tier.t \ ./tests/basic/tier/fops-during-migration.t \ ./tests/basic/tier/record-metadata-heat.t \ - ./tests/basic/tier/tier-snapshot.t \ + ./tests/basic/tier/tier-snapshot.t \ ./tests/bugs/snapshot/bug-1109889.t \ ./tests/bugs/distribute/bug-1066798.t \ ./tests/bugs/glusterd/bug-1238706-daemons-stop-on-peer-cleanup.t \ @@ -183,7 +185,16 @@ function is_bad_test () ; do [ x"$name" = x"$bt" ] && return 0 # bash: zero means true/success done - return 1 # bash: non-zero means false/failure + return 1 # bash: non-zero means false/failure +} + +function is_unsupported_test() +{ + if [ x"$OSTYPE" != x"NetBSD" ]; then + return 1 + fi + + grep -iqs tier $1 } function run_tests() @@ -197,16 +208,22 @@ function run_tests() for t in $(find ${regression_testsdir}/tests -name '*.t' \ | LC_COLLATE=C sort) ; do - old_cores=$(ls /core.* 2> /dev/null | wc -l) + old_cores=$(ls /core.* 2> /dev/null | wc -l) if match $t "$@" ; then echo echo "==================================================" - if is_bad_test $t; then + if is_bad_test $t; then echo "Skipping bad test file $t" echo "==================================================" echo continue fi + if is_unsupported_test $t; then + echo "Skipping test file $t (feature unsupported on platform)" + echo "==================================================" + echo + continue + fi echo "Running tests in file $t" starttime="$(date +%s)" prove -mf --timer $t -- cgit