LU-4093 tests: fix typo and inverted test in original patch

1st/original patch for LU-4093 ticket, patch-set #3 of Change at
http://review.whamcloud.com/8157/ with Id
I3274e5864469251e6f2040377b84429e4f218f47, contains a typo and
an inverted test that prevent it to work as expected but rather
with reversed logic.

Signed-off-by: Bruno Faccini <bruno.faccini@intel.com>
Change-Id: I3f8031b85db116711312b8f7decf2c33761b89e6
Reviewed-on: http://review.whamcloud.com/8329
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Jian Yu <jian.yu@intel.com>
Reviewed-by: Nathaniel Clark <nathaniel.l.clark@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>

diff --git a/lustre/tests/sanity-hsm.sh b/lustre/tests/sanity-hsm.sh
index 348f94d..d4ba709 100644 (file)
--- a/lustre/tests/sanity-hsm.sh
+++ b/lustre/tests/sanity-hsm.sh
@@ -178,9 +178,9 @@ copytool_cleanup() {
                idx=$(($mdtno - 1))
                mdt_hsmctrl="mdt.$FSNAME-MDT000${idx}.hsm_control"
                oldstate=$(do_facet mds${mdtno} "$LCTL get_param -n " \
-                                  "$MDT_HSMCTRL")
+                                  "$mdt_hsmctrl")
                # skip already stop[ed,ing] CDTs
-               echo $oldstate | grep stop || continue
+               echo $oldstate | grep stop && continue
 
                do_facet mds${mdtno} "$LCTL set_param $mdt_hsmctrl=shutdown"
                wait_result mds${mdtno} "$LCTL get_param -n $mdt_hsmctrl" \