Whamcloud - gitweb
LU-13813 tests: fix stack_trap in conf-sanity test 110/111 22/48022/4
authorJian Yu <yujian@whamcloud.com>
Sat, 23 Jul 2022 07:27:43 +0000 (00:27 -0700)
committerOleg Drokin <green@whamcloud.com>
Fri, 19 Aug 2022 04:33:01 +0000 (04:33 +0000)
This patch fixes stack_trap in conf-sanity test 110 and 111
to restore test environment.

Test-Parameters: trivial env=SLOW=yes,ENABLE_QUOTA=yes,CONF_SANITY_EXCEPT=107 \
clientdistro=el8.5 serverdistro=el8.5 testlist=conf-sanity

Test-Parameters: trivial env=SLOW=yes,ENABLE_QUOTA=yes \
fstype=zfs \
clientdistro=el8.5 serverdistro=el8.5 testlist=conf-sanity

Test-Parameters: trivial env=SLOW=yes,ENABLE_QUOTA=yes,CONF_SANITY_EXCEPT=107 \
mdscount=2 mdtcount=4 \
clientdistro=el8.5 serverdistro=el8.5 testlist=conf-sanity

Change-Id: I540d96e8ad2c4990e7da18fe22256b44e9a19c72
Signed-off-by: Jian Yu <yujian@whamcloud.com>
Reviewed-on: https://review.whamcloud.com/48022
Tested-by: jenkins <devops@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Arshad Hussain <arshad.hussain@aeoncomputing.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
lustre/tests/conf-sanity.sh

index acf020d..b28a7c5 100644 (file)
@@ -8738,7 +8738,7 @@ test_110()
        local old_mdscount=$MDSCOUNT
        local old_ostcount=$OSTCOUNT
        local replace=""
-       stack_trap "MDSCOUNT=$old_mdscount OSTCOUNT=$old_ostcount" EXIT
+       stack_trap "MDSCOUNT=$old_mdscount OSTCOUNT=$old_ostcount; reformat"
        MDSCOUNT=1
        OSTCOUNT=1
 
@@ -8845,7 +8845,8 @@ test_111() {
        local old_ostcount=$OSTCOUNT
        local old_mdssize=$MDSSIZE
        local replace=""
-       stack_trap "MDSSIZE=$MDSSIZE MDSCOUNT=$MDSCOUNT OSTCOUNT=$OSTCOUNT" EXIT
+       stack_trap "MDSSIZE=$old_mdssize MDSCOUNT=$old_mdscount \
+                   OSTCOUNT=$old_ostcount; reformat"
        MDSCOUNT=1
        OSTCOUNT=1
        (( MDSSIZE < 2400000 )) && MDSSIZE=2400000 # need at least 2.4GB
@@ -8941,7 +8942,8 @@ test_111() {
        umount_client $MOUNT2 -f
        cleanup
 
-       (( $needskip )) && skip "ETA ${left}s after $num files / ${taken}s is too long"
+       ! (( $needskip )) ||
+               echo "ETA ${left}s after $num files / ${taken}s is too long"
 
        run_e2fsck $(facet_active_host mds1) $(mdsdevname 1) -n
        MDSCOUNT=$old_mdscount