From: Bruno Faccini <bruno.faccini@intel.com>
Date: Tue, 19 Nov 2013 11:00:51 +0000 (+0100)
Subject: LU-4093 tests: fix typo and inverted test in original patch
X-Git-Tag: 2.5.1-RC1~102
X-Git-Url: https://git.whamcloud.com/?a=commitdiff_plain;h=e5280accb58515fb8966c0e50766d9c989078d24;p=fs%2Flustre-release.git

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 00bee8e..b999f20 100644
--- a/lustre/tests/sanity-hsm.sh
+++ b/lustre/tests/sanity-hsm.sh
@@ -179,9 +179,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" \