Whamcloud - gitweb
LU-2123 tests: sanity 57 and 129 can't detect fstype
authorJames Simmons <uja.ornl@gmail.com>
Tue, 9 Oct 2012 14:36:12 +0000 (10:36 -0400)
committerOleg Drokin <green@whamcloud.com>
Sat, 13 Oct 2012 23:24:13 +0000 (19:24 -0400)
In sanity test 57a,57b and 129 the function facet_type_fstype
is used to determine the file system backend. This function
does not actually exist so this test never run for the
ldiskfs case. The fix is to use the proper function which
is facet_fstype.

Signed-off-by: James Simmons <uja.ornl@gmail.com>
Change-Id: Ia9766a3a2200c6ce2d48ff0265eb73a4d71c06e7
Reviewed-on: http://review.whamcloud.com/4230
Tested-by: Hudson
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Li Wei <liwei@whamcloud.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Yu Jian <yujian@whamcloud.com>
lustre/tests/sanity.sh

index 27970e0..d38300b 100644 (file)
@@ -4012,7 +4012,7 @@ run_test 56w "check lfs_migrate -c stripe_count works"
 
 test_57a() {
        # note test will not do anything if MDS is not local
 
 test_57a() {
        # note test will not do anything if MDS is not local
-       if [ "$(facet_type_fstype MDS)" != ldiskfs ]; then
+       if [ "$(facet_fstype $SINGLEMDS)" != ldiskfs ]; then
                skip "Only applicable to ldiskfs-based MDTs"
                return
        fi
                skip "Only applicable to ldiskfs-based MDTs"
                return
        fi
@@ -4032,7 +4032,7 @@ test_57a() {
 run_test 57a "verify MDS filesystem created with large inodes =="
 
 test_57b() {
 run_test 57a "verify MDS filesystem created with large inodes =="
 
 test_57b() {
-       if [ "$(facet_type_fstype MDS)" != ldiskfs ]; then
+       if [ "$(facet_fstype $SINGLEMDS)" != ldiskfs ]; then
                skip "Only applicable to ldiskfs-based MDTs"
                return
        fi
                skip "Only applicable to ldiskfs-based MDTs"
                return
        fi
@@ -7093,7 +7093,7 @@ set_dir_limits () {
        done
 }
 test_129() {
        done
 }
 test_129() {
-       if [ "$(facet_type_fstype MDS)" != ldiskfs ]; then
+       if [ "$(facet_fstype $SINGLEMDS)" != ldiskfs ]; then
                skip "Only applicable to ldiskfs-based MDTs"
                return
        fi
                skip "Only applicable to ldiskfs-based MDTs"
                return
        fi