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

Update zfs_admin_snapshot value (disabled) 

It's disabled by default, update code and tests to reflect
the documentation.

Minor cleanup in delegate_common.kshlib.

Reviewed-by: Gregor Kopka <gregor@kopka.net>
Reviewed-by: John Kennedy <john.kennedy@delphix.com>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: George Melikov <mail@gmelikov.ru>
Closes #7835 
Closes #8045
This commit is contained in:
George Melikov 2018-11-09 03:17:12 +03:00 committed by Brian Behlendorf
parent d8244d34bd
commit 877d925a9e
6 changed files with 30 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
module/zfs/zfs_ctldir.c
View File

@ -29,6 +29,7 @@
* Brian Behlendorf <behlendorf1@llnl.gov>
* Copyright (c) 2013 by Delphix. All rights reserved.
* Copyright 2015, OmniTI Computer Consulting, Inc. All rights reserved.
* Copyright (c) 2018 George Melikov. All Rights Reserved.
*/
/*
@ -106,7 +107,7 @@ static krwlock_t zfs_snapshot_lock;
* Control Directory Tunables (.zfs)
*/
int zfs_expire_snapshot = ZFSCTL_EXPIRE_SNAPSHOT;
int zfs_admin_snapshot = 1;
int zfs_admin_snapshot = 0;
typedef struct {
char *se_name; /* full snapshot name */

7
tests/zfs-tests/tests/functional/delegate/cleanup.ksh
View File

@ -27,6 +27,7 @@
#
# Copyright (c) 2013, 2016 by Delphix. All rights reserved.
# Copyright (c) 2018 George Melikov. All Rights Reserved.
#
. $STF_SUITE/include/libtest.shlib
@ -42,6 +43,8 @@ if ! is_linux; then
fi
fi
default_cleanup
if is_linux; then
log_must set_tunable64 zfs_admin_snapshot 0
fi
log_pass
default_cleanup

17
tests/zfs-tests/tests/functional/delegate/delegate_common.kshlib
View File

@ -27,6 +27,7 @@
#
# Copyright (c) 2013, 2016 by Delphix. All rights reserved.
# Copyright 2016 Nexenta Systems, Inc.
# Copyright (c) 2018 George Melikov. All Rights Reserved.
#
. $STF_SUITE/include/libtest.shlib
@ -766,7 +767,7 @@ function verify_fs_clone
typeset fs=$3
typeset stamp=${perm}.${user}.$(date +'%F-%T-%N')
typeset basefs=${fs%/*}
typeset basefs=${fs%/*}
typeset snap=$fs@snap.$stamp
typeset clone=$basefs/cfs.$stamp
@ -811,7 +812,7 @@ function verify_fs_rename
typeset fs=$3
typeset stamp=${perm}.${user}.$(date +'%F-%T-%N')
typeset basefs=${fs%/*}
typeset basefs=${fs%/*}
typeset snap=$fs@snap.$stamp
typeset renamefs=$basefs/nfs.$stamp
@ -1001,7 +1002,7 @@ function verify_fs_promote
typeset fs=$3
typeset stamp=${perm}.${user}.$(date +'%F-%T-%N')
typeset basefs=${fs%/*}
typeset basefs=${fs%/*}
typeset snap=$fs@snap.$stamp
typeset clone=$basefs/cfs.$stamp
@ -1368,7 +1369,7 @@ function verify_vol_snapshot
typeset vol=$3
typeset stamp=${perm}.${user}.$(date +'%F-%T-%N')
typeset basevol=${vol%/*}
typeset basevol=${vol%/*}
typeset snap=$vol@snap.$stamp
user_run $user zfs snapshot $snap
@ -1393,7 +1394,7 @@ function verify_vol_rollback
typeset vol=$3
typeset stamp=${perm}.${user}.$(date+'%F-%T-%N')
typeset basevol=${vol%/*}
typeset basevol=${vol%/*}
typeset snap=$vol@snap.$stamp
typeset oldval
@ -1428,7 +1429,7 @@ function verify_vol_clone
typeset vol=$3
typeset stamp=${perm}.${user}.$(date+'%F-%T-%N')
typeset basevol=${vol%/*}
typeset basevol=${vol%/*}
typeset snap=$vol@snap.$stamp
typeset clone=$basevol/cvol.$stamp
@ -1474,7 +1475,7 @@ function verify_vol_rename
typeset vol=$3
typeset stamp=${perm}.${user}.$(date+'%F-%T-%N')
typeset basevol=${vol%/*}
typeset basevol=${vol%/*}
typeset snap=$vol@snap.$stamp
typeset clone=$basevol/cvol.$stamp
typeset renamevol=$basevol/nvol.$stamp
@ -1521,7 +1522,7 @@ function verify_vol_promote
typeset vol=$3
typeset stamp=${perm}.${user}.$(date+'%F-%T-%N')
typeset basevol=${vol%/*}
typeset basevol=${vol%/*}
typeset snap=$vol@snap.$stamp
typeset clone=$basevol/cvol.$stamp

9
tests/zfs-tests/tests/functional/delegate/setup.ksh
View File

@ -27,6 +27,7 @@
#
# Copyright (c) 2013, 2016 by Delphix. All rights reserved.
# Copyright (c) 2018 George Melikov. All Rights Reserved.
#
. $STF_SUITE/include/libtest.shlib
@ -68,7 +69,9 @@ if [ $? -ne 0 ]; then
fi
DISK=${DISKS%% *}
default_volume_setup $DISK
log_must chmod 777 $TESTDIR
log_pass
if is_linux; then
log_must set_tunable64 zfs_admin_snapshot 1
fi
default_volume_setup $DISK

4
tests/zfs-tests/tests/functional/snapshot/cleanup.ksh
View File

@ -31,4 +31,8 @@
. $STF_SUITE/include/libtest.shlib
if is_linux; then
log_must set_tunable64 zfs_admin_snapshot 0
fi
default_container_cleanup

4
tests/zfs-tests/tests/functional/snapshot/setup.ksh
View File

@ -33,4 +33,8 @@
DISK=${DISKS%% *}
if is_linux; then
log_must set_tunable64 zfs_admin_snapshot 1
fi
default_container_volume_setup ${DISK}