mirror of
https://git.proxmox.com/git/mirror_zfs.git
synced 2026-05-22 02:27:36 +03:00
OpenZFS 9164 - assert: newds == os->os_dsl_dataset
Authored by: Andriy Gapon <avg@FreeBSD.org> Reviewed by: Matt Ahrens <mahrens@delphix.com> Reviewed by: Don Brady <don.brady@delphix.com> Reviewed-by: loli10K <ezomori.nozomu@gmail.com> Reviewed-by: Tony Hutter <hutter2@llnl.gov> Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov> Approved by: Richard Lowe <richlowe@richlowe.net> Ported-by: Giuseppe Di Natale <dinatale2@llnl.gov> Porting Notes: * Re-enabled and tweaked the zpool_upgrade_007_pos test case to successfully run in under 5 minutes. OpenZFS-issue: https://www.illumos.org/issues/9164 OpenZFS-commit: https://github.com/openzfs/openzfs/commit/0e776dc06a Closes #6112 Closes #7336
This commit is contained in:
committed by
Brian Behlendorf
parent
99f505a4d7
commit
5e00213e43
@@ -42,11 +42,22 @@ set -A spa_create_versions 9 15 24
|
||||
set -A zpl_upgrade_versions 3 4 5
|
||||
set -A spa_upgrade_versions 9 15 24
|
||||
|
||||
function get_pool_version #pool
|
||||
{
|
||||
ver=$(get_pool_prop version $1)
|
||||
|
||||
if [ "$ver" = "-" ]; then
|
||||
ver="5000"
|
||||
fi
|
||||
|
||||
echo "$ver"
|
||||
}
|
||||
|
||||
function default_setup_datasets #rootfs
|
||||
{
|
||||
typeset rootfs=$1
|
||||
typeset pool=${rootfs%%/*}
|
||||
typeset -i vp=$(get_pool_prop version $pool)
|
||||
typeset -i vp=$(get_pool_version $pool)
|
||||
typeset -i version
|
||||
typeset -i m
|
||||
typeset -i spa_version
|
||||
@@ -107,7 +118,7 @@ function default_cleanup_datasets #rootfs
|
||||
typeset rootfs=$1
|
||||
|
||||
if datasetexists $rootfs ; then
|
||||
log_must zfs destroy -Rf $rootfs
|
||||
log_must_busy zfs destroy -Rf $rootfs
|
||||
fi
|
||||
|
||||
if datasetnonexists $rootfs ; then
|
||||
@@ -119,7 +130,7 @@ function default_check_zfs_upgrade #rootfs
|
||||
{
|
||||
typeset rootfs=$1
|
||||
typeset pool=${rootfs%%/*}
|
||||
typeset -i vp=$(get_pool_prop version $pool)
|
||||
typeset -i vp=$(get_pool_version $pool)
|
||||
typeset -i m
|
||||
typeset -i spa_version
|
||||
typeset -i zpl_version
|
||||
@@ -150,9 +161,9 @@ function default_check_zfs_upgrade #rootfs
|
||||
|
||||
if (( df_ret != 0 )); then
|
||||
if (( spa_version != 0 )) && (( vp < spa_version )); then
|
||||
log_mustnot zfs upgrade $opt -a >/dev/null
|
||||
log_must zpool upgrade $pool >/dev/null
|
||||
vp=$(get_pool_prop version $pool)
|
||||
log_mustnot eval "zfs upgrade $opt -a >/dev/null"
|
||||
log_must eval "zpool upgrade $pool >/dev/null"
|
||||
vp=$(get_pool_version $pool)
|
||||
fi
|
||||
|
||||
log_must zfs upgrade $opt -a
|
||||
|
||||
@@ -93,9 +93,8 @@ function destroy_upgraded_pool
|
||||
typeset -n pool_files=ZPOOL_VERSION_${vers}_FILES
|
||||
typeset -n pool_name=ZPOOL_VERSION_${vers}_NAME
|
||||
|
||||
if poolexists $pool_name; then
|
||||
log_must zpool destroy $pool_name
|
||||
fi
|
||||
destroy_pool $pool_name
|
||||
|
||||
for file in $pool_files; do
|
||||
rm -f $TEST_BASE_DIR/$file
|
||||
done
|
||||
|
||||
@@ -48,19 +48,18 @@
|
||||
|
||||
verify_runnable "global"
|
||||
|
||||
https://github.com/zfsonlinux/zfs/issues/6112
|
||||
log_unsupported "Known issue #6112"
|
||||
|
||||
function cleanup
|
||||
{
|
||||
destroy_upgraded_pool $config
|
||||
}
|
||||
|
||||
POOL_CONFIGS="1raidz 1mirror 2raidz 2mirror 3raidz 3mirror"
|
||||
|
||||
log_assert "Import pools of all versions - 'zfs upgrade' on each pool works"
|
||||
log_onexit cleanup
|
||||
|
||||
# $CONFIGS gets set in the .cfg script
|
||||
for config in $CONFIGS; do
|
||||
for config in $POOL_CONFIGS; do
|
||||
typeset -n pool_name=ZPOOL_VERSION_${config}_NAME
|
||||
|
||||
create_old_pool $config
|
||||
|
||||
Reference in New Issue
Block a user