Archive-Team
/
zfs
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 103 Wiki Activity

ZTS: zpool_split_indirect deletes zfstest log file 

The cleanup routine for this test attempts to remove some temporary
files with `rm -f $VDEV_*`, but VDEV_ is undefined. As a result, all
files in the current working directory (/var/tmp/test_results/current)
get removed instead. This includes the complete log file of all tests.

Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: George Melikov <mail@gmelikov.ru>
Reviewed-by: George Amanakis <gamanakis@gmail.com>
Reviewed-by: Ryan Moeller <ryan@iXsystems.com>
Signed-off-by: John Kennedy <john.kennedy@delphix.com>
Closes #10324
This commit is contained in:
John Wren Kennedy 2020-05-14 10:39:47 -06:00 committed by GitHub
parent 41035a0496
commit e1fcd940e7
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
tests/zfs-tests/tests/functional/cli_root/zpool_split/zpool_split_indirect.ksh
View File

@ -16,6 +16,7 @@
# #
# Copyright (c) 2020, George Amanakis. All rights reserved. # Copyright (c) 2020, George Amanakis. All rights reserved.
# Copyright (c) 2020 by Delphix. All rights reserved.
# #
. $STF_SUITE/include/libtest.shlib . $STF_SUITE/include/libtest.shlib
@ -42,7 +43,7 @@ function cleanup
if poolexists $TESTPOOL2 ; then if poolexists $TESTPOOL2 ; then
destroy_pool $TESTPOOL2 destroy_pool $TESTPOOL2
fi fi
rm -f $VDEV_* rm -f $VDEV_TEMP $VDEV_M1 $VDEV_M2
} }
log_onexit cleanup log_onexit cleanup