Whamcloud - gitweb
LU-13813 tests: fix stack_trap in conf-sanity test 110/111 21/48021/3
authorJian Yu <yujian@whamcloud.com>
Sat, 23 Jul 2022 06:49:40 +0000 (23:49 -0700)
committerOleg Drokin <green@whamcloud.com>
Sat, 20 Aug 2022 16:14:13 +0000 (16:14 +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 \
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 \
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/48021
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
lustre/tests/conf-sanity.sh [changed mode: 0644->0755]

old mode 100644 (file)
new mode 100755 (executable)
index 81c9a1e..0a549ae
@@ -8641,7 +8641,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
 
@@ -8748,7 +8748,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
@@ -8844,7 +8845,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