c/mirror_zfs
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_zfs.git synced 2024-12-25 18:59:33 +03:00
Code Issues Packages Projects Releases Wiki Activity

ZTS: harden xattr/cleanup.ksh

Browse Source 
When the xattr/cleanup.ksh script is unable to remove the test group
due to an active process then it will not call default_cleanup.  This
will result in a zvol_ENOSPC/setup failure when attempting to create
the /mnt/testdir directory which will already exist.

Resolve the issue by performing the default_cleanup before removing
the test user and group to ensure this step always happens.  Also
allow one more retry to further minimize the likelihood of the
cleanup failing.

Reviewed-by: Ryan Moeller <ryan@ixsystems.com>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes #9358
This commit is contained in:
Brian Behlendorf 2019-09-25 09:24:45 -07:00 committed by GitHub
parent f81d5ef686
commit 5e5cefbaee
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tests/zfs-tests/include/libtest.shlib
View File

@ -2375,7 +2375,7 @@ function del_user #<logname> <basedir>
fi
if id $user > /dev/null 2>&1; then
log_must_retry "currently used" 5 userdel $user
log_must_retry "currently used" 6 userdel $user
fi
[[ -d $basedir/$user ]] && rm -fr $basedir/$user

10
tests/zfs-tests/tests/functional/xattr/cleanup.ksh
View File

@ -30,9 +30,6 @@
. $STF_SUITE/include/libtest.shlib
. $STF_SUITE/tests/functional/xattr/xattr_common.kshlib
del_user $ZFS_USER
del_group $ZFS_GROUP
USES_NIS=$(cat $TEST_BASE_DIR/zfs-xattr-test-nis.txt)
rm $TEST_BASE_DIR/zfs-xattr-test-nis.txt
@ -41,4 +38,9 @@ then
svcadm enable svc:/network/nis/client:default
fi
default_cleanup
default_cleanup_noexit
del_user $ZFS_USER
del_group $ZFS_GROUP
log_pass