From: Hongchao Zhang <hongchao.zhang@intel.com>
Date: Fri, 17 Nov 2017 15:05:17 +0000 (+0800)
Subject: LU-8999 test: check chown result
X-Git-Tag: 2.10.3-RC1~11
X-Git-Url: https://git.whamcloud.com/?a=commitdiff_plain;h=3a0fc1a8d65d4796a3da895006dcc5012e9e1b3f;p=fs%2Flustre-release.git

LU-8999 test: check chown result

For test_38 in sanity-quota, the chown operation should be
checked whether it succeeds or not. the patch also collects
the output of quota users if the test failed.

Lustre-change: https://review.whamcloud.com/30243
Lustre-commit: 817b367707d0d4f5c406b4f18cffceea2aae8406

Change-Id: I6c04f9519f3f097af7126064380faf1bdc4fff6a
Signed-off-by: Hongchao Zhang <hongchao.zhang@intel.com>
Reviewed-by: Fan Yong <fan.yong@intel.com>
Reviewed-by: Jian Yu <jian.yu@intel.com>
Signed-off-by: Minh Diep <minh.diep@intel.com>
Reviewed-on: https://review.whamcloud.com/30631
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
---

diff --git a/lustre/tests/sanity-quota.sh b/lustre/tests/sanity-quota.sh
index a099996..73acbf0 100755
--- a/lustre/tests/sanity-quota.sh
+++ b/lustre/tests/sanity-quota.sh
@@ -2732,7 +2732,8 @@ test_38() {
 	echo "Create $file_cnt files..."
 	for i in `seq $file_cnt`; do
 		touch $TESTFILE-$i
-		chown $((file_cnt - i)):$((file_cnt - i)) $TESTFILE-$i
+		chown $((file_cnt - i)):$((file_cnt - i)) $TESTFILE-$i ||
+			error "failed to chown $TESTFILE-$i"
 	done
 	cancel_lru_locks osc
 	sync; sync_all_data || true
@@ -2744,7 +2745,10 @@ test_38() {
 	acct_cnt=$(do_facet mds1 $LCTL get_param $procf | grep "id:" | wc -l)
 	echo "Found $acct_cnt id entries"
 
-	[ $file_cnt -eq $acct_cnt ] || error "skipped id entries"
+	[ $file_cnt -eq $acct_cnt ] || {
+		do_facet mds1 $LCTL get_param $procf
+		error "skipped id entries"
+	}
 
 	cleanup_quota_test
 }