c/mirror_zfs
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_zfs.git synced 2026-05-22 02:27:36 +03:00
Code Issues Packages Projects Releases Wiki Activity

fix(mount): do not truncate shares not zfs mount

Browse Source 
When running zfs share -a resetting the exports.d/zfs.exports makes
sense the get a clean state.
Truncating was also called with zfs mount which would not populate the
file again.
Add test to verify shares persist after mount -a.

Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Stefan Lendl <s.lendl@proxmox.com>
Closes #15607 
Closes #15660
This commit is contained in:
Stefan Lendl
2024-01-12 21:05:11 +01:00
committed by GitHub
parent c4fa674367
commit 66670ba9f0
4 changed files with 67 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/zfs-tests/tests/Makefile.am
+1
View File
@@ -912,6 +912,7 @@ nobase_dist_datadir_zfs_tests_tests_SCRIPTS += \
functional/cli_root/zfs_share/zfs_share_012_pos.ksh \
functional/cli_root/zfs_share/zfs_share_013_pos.ksh \
functional/cli_root/zfs_share/zfs_share_concurrent_shares.ksh \
functional/cli_root/zfs_share/zfs_share_after_mount.ksh \
functional/cli_root/zfs_snapshot/cleanup.ksh \
functional/cli_root/zfs_snapshot/setup.ksh \
functional/cli_root/zfs_snapshot/zfs_snapshot_001_neg.ksh \
tests/zfs-tests/tests/functional/cli_root/zfs_share/zfs_share_after_mount.ksh
Executable
+62
View File
@@ -0,0 +1,62 @@
#!/bin/ksh -p
#
# CDDL HEADER START
#
# The contents of this file are subject to the terms of the
# Common Development and Distribution License (the "License").
# You may not use this file except in compliance with the License.
#
# You can obtain a copy of the license at usr/src/OPENSOLARIS.LICENSE
# or https://opensource.org/licenses/CDDL-1.0.
# See the License for the specific language governing permissions
# and limitations under the License.
#
# When distributing Covered Code, include this CDDL HEADER in each
# file and include the License file at usr/src/OPENSOLARIS.LICENSE.
# If applicable, add the following below this CDDL HEADER, with the
# fields enclosed by brackets "[]" replaced with your own identifying
# information: Portions Copyright [yyyy] [name of copyright owner]
#
# CDDL HEADER END
#
#
# Copyright (c) 2023 by Proxmox. All rights reserved.
#
. $STF_SUITE/include/libtest.shlib
# DESCRIPTION:
# Verify that nfs shares persist after zfs mount -a
#
# STRATEGY:
# 1. Verify that the filesystem is not shared.
# 2. Enable the 'sharenfs' property
# 3. Verify filesystem is shared
# 4. Invoke 'zfs mount -a'
# 5. Verify filesystem is still shared
verify_runnable "global"
function cleanup
{
log_must zfs set sharenfs=off $TESTPOOL/$TESTFS
is_shared $TESTPOOL/$TESTFS && \
log_must unshare_fs $TESTPOOL/$TESTFS
log_must zfs share -a
}
log_onexit cleanup
cleanup
log_must zfs set sharenfs="on" $TESTPOOL/$TESTFS
log_must is_shared $TESTPOOL/$TESTFS
log_must is_exported $TESTPOOL/$TESTFS
log_must zfs mount -a
log_must is_shared $TESTPOOL/$TESTFS
log_must is_exported $TESTPOOL/$TESTFS
log_pass "Verify that nfs shares persist after zfs mount -a"