Whamcloud - gitweb
LU-3879 test: Check for e2fsprogs on server 50/8350/2
authorJames Nunez <james.a.nunez@intel.com>
Wed, 20 Nov 2013 20:30:37 +0000 (13:30 -0700)
committerOleg Drokin <oleg.drokin@intel.com>
Sat, 23 Nov 2013 13:45:58 +0000 (13:45 +0000)
sanity-quota checks if e2fsprogs is loaded on the client,
but should be checking for e2fsprogs on the server. This
patch changes checking for dumpe2fs on the client to
checking on the MDS.

Signed-off-by: James Nunez <james.a.nunez@intel.com>
Change-Id: I85d3b71dedb6e33775c7823b11f18c7ca405e858
Reviewed-on: http://review.whamcloud.com/8350
Tested-by: Jenkins
Reviewed-by: Niu Yawei <yawei.niu@intel.com>
Reviewed-by: Jian Yu <jian.yu@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lustre/tests/sanity-quota.sh

index 2cfc409..6622cd3 100644 (file)
@@ -43,9 +43,10 @@ if [ $(lustre_version_code $SINGLEMDS) -lt $(version_code 2.3.50) ]; then
        exec $LUSTRE/tests/sanity-quota-old.sh
 fi
 
-# if e2fsprogs support quota feature?
-if [ $(facet_fstype $SINGLEMDS) == ldiskfs ] && \
-       ! $DEBUGFS -c -R supported_features | grep -q 'quota'; then
+# Does e2fsprogs support quota feature?
+if [ $(facet_fstype $SINGLEMDS) == ldiskfs ] &&
+       do_facet $SINGLEMDS "! $DEBUGFS -c -R supported_features |
+               grep -q 'quota'"; then
        skip "e2fsprogs doesn't support quota" && exit 0
 fi