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 (executable)
--- 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
 }