From: Saurabh Tandan <saurabh.tandan@intel.com>
Date: Thu, 13 Apr 2017 18:08:44 +0000 (-0700)
Subject: LU-9318 test: sanity-lsnapshot skipped for all non-zfs configs
X-Git-Tag: 2.9.56~4
X-Git-Url: https://git.whamcloud.com/?a=commitdiff_plain;h=eb3379162c9c3044759013390cdba7b53783b31d;p=fs%2Flustre-release.git

LU-9318 test: sanity-lsnapshot skipped for all non-zfs configs

The problem with the script was that it tried to check
for ZFS file system after the script tried to add snapshot
to the debug list.

As part of solution the script just needs to check for ZFS
file system or the version is too old at the start for of
the script.

Test-Parameters: trivial testlist=sanity-lsnapshot,sanity-lsnapshot,sanity-lsnapshot
Signed-off-by: Saurabh Tandan <saurabh.tandan@intel.com>
Change-Id: Ie59dac7543d1e35b924c84d19c34a5ec5b903a14
Reviewed-on: https://review.whamcloud.com/26600
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: James Nunez <james.a.nunez@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
---

diff --git a/lustre/tests/sanity-lsnapshot.sh b/lustre/tests/sanity-lsnapshot.sh
old mode 100644
new mode 100755
index 985cf99..bb699c2
--- a/lustre/tests/sanity-lsnapshot.sh
+++ b/lustre/tests/sanity-lsnapshot.sh
@@ -17,6 +17,13 @@ init_test_env $@
 . ${CONFIG:=$LUSTRE/tests/cfg/$NAME.sh}
 init_logging
 
+[[ $(lustre_version_code mds1) -lt $(version_code 2.9.55) ]] ||
+[[ $(lustre_version_code ost1) -lt $(version_code 2.9.55) ]] &&
+	skip "Need server version at least 2.9.55" && exit 0
+[[ $(facet_fstype mds1) = "ldiskfs" ]] ||
+[[ $(facet_fstype ost1) = "ldiskfs" ]] &&
+	skip "can't test snapshots with ldiskfs" && exit 0
+
 require_dsh_mds || exit 0
 require_dsh_ost || exit 0