Whamcloud - gitweb
LU-9474 tests: remove unneeded hsm_set_param for raolu tests 29/31429/3
authorQuentin Bouget <quentin.bouget@cea.fr>
Tue, 27 Feb 2018 15:22:08 +0000 (15:22 +0000)
committerOleg Drokin <oleg.drokin@intel.com>
Mon, 21 May 2018 16:51:53 +0000 (16:51 +0000)
A previous commit introduced the stack_trap() utility function,
the patch should have removed any duplicate cleanup actions but
missed a few in test_26{a,b,c,d} of sanity-hsm.

This commits removes them.

Test-Parameters: trivial testlist=sanity-hsm
Signed-off-by: Quentin Bouget <quentin.bouget@cea.fr>
Change-Id: I63e1df6610d0fbceb2f88a5b59b8263b8ccaf525
Reviewed-on: https://review.whamcloud.com/31429
Tested-by: Jenkins
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: James Nunez <james.a.nunez@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
lustre/tests/sanity-hsm.sh

index 50c0e26..41dd17f 100755 (executable)
@@ -2504,8 +2504,6 @@ test_26a() {
 
        rm -f $f2
 
-       set_hsm_param remove_archive_on_last_unlink 0
-
        wait_request_state $fid2 REMOVE SUCCEED
 
        assert_request_count $fid REMOVE 0 \
@@ -2534,8 +2532,6 @@ test_26b() {
 
        rm -f $f
 
-       set_hsm_param remove_archive_on_last_unlink 0
-
        wait_request_state $fid REMOVE WAITING
 
        cdt_enable
@@ -2592,8 +2588,6 @@ test_26c() {
        # signal, close file, and exit ...
        wait $pid || error "wait PID $PID failed"
 
-       set_hsm_param remove_archive_on_last_unlink 0
-
        wait_request_state $fid REMOVE SUCCEED
 }
 run_test 26c "RAoLU effective when file closed"
@@ -2627,8 +2621,6 @@ test_26d() {
 
        mds_evict_client
 
-       set_hsm_param remove_archive_on_last_unlink 0
-
        wait_request_state $fid REMOVE SUCCEED
 
        client_up || client_up || true