Whamcloud - gitweb
LU-10316 tests: skip checksum check for sanity 77c 02/30402/7
authorJames Nunez <james.a.nunez@intel.com>
Wed, 6 Dec 2017 00:17:16 +0000 (17:17 -0700)
committerOleg Drokin <oleg.drokin@intel.com>
Tue, 9 Jan 2018 05:36:10 +0000 (05:36 +0000)
sanity test 77c does not need to verify the checksum for
some versions of Lustre servers. The code to skip the
checksum verification is in place, but it is missing a
'retun 0;' to exit the test.

Test-Parameters: trivial
Signed-off-by: James Nunez <james.a.nunez@intel.com>
Change-Id: I9ea19f699d1142475f84d59b9c31880b7daf7f52
Reviewed-on: https://review.whamcloud.com/30402
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Cliff White <cliff.white@intel.com>
lustre/tests/sanity.sh

index 26183a2..45cb09b 100755 (executable)
@@ -6664,7 +6664,7 @@ test_77c() {
        [[ "$orig_cksum" == "$dump_cksum" ]] ||
                error "dump content does not match on Client"
 
        [[ "$orig_cksum" == "$dump_cksum" ]] ||
                error "dump content does not match on Client"
 
-       $check_ost || skip "No need to check cksum dump on OSS"
+       $check_ost || { skip "No need to check cksum dump on OSS"; return 0; }
 
        # check cksum dump on OSS
        ost_file=$(do_facet ost1 ls ${ost_file_prefix}\*)
 
        # check cksum dump on OSS
        ost_file=$(do_facet ost1 ls ${ost_file_prefix}\*)