aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLee Trager <lee@trager.us>2023-08-18 01:11:56 -0700
committerZorro Lang <zlang@kernel.org>2023-08-19 03:36:22 +0800
commiteccdeae8fc417ecb369b822a8182fd5eeab751e1 (patch)
tree804dd60025fe97593d3adc1816a0471bf463287d
parent697988c94e870a2e9be9b0827078e718ae20995f (diff)
downloadxfstests-dev-eccdeae8fc417ecb369b822a8182fd5eeab751e1.tar.gz
fstests: Verify dir permissions when creating a stub subvolume
btrfs supports creating nesting subvolumes however snapshots are not recurive. When a snapshot is taken of a volume which contains a subvolume the subvolume is replaced with a stub subvolume which has the same name and uses inode number 2. This test validates that the stub volume copies permissions of the original volume. Signed-off-by: Lee Trager <lee@trager.us> Reviewed-by: Zorro Lang <zlang@redhat.com> Reviewed-by: Filipe Manana <fdmanana@suse.com> Signed-off-by: Zorro Lang <zlang@kernel.org>
-rw-r--r--common/rc5
-rwxr-xr-xtests/btrfs/30052
-rw-r--r--tests/btrfs/300.out18
-rwxr-xr-xtests/overlay/0207
4 files changed, 76 insertions, 6 deletions
diff --git a/common/rc b/common/rc
index 66d270acf0..7005f4bf62 100644
--- a/common/rc
+++ b/common/rc
@@ -5224,6 +5224,11 @@ _soak_loop_running() {
return 0
}
+_require_unshare() {
+ unshare -f -r -m -p -U $@ true &>/dev/null || \
+ _notrun "unshare: command not found, should be in util-linux"
+}
+
init_rc
################################################################################
diff --git a/tests/btrfs/300 b/tests/btrfs/300
new file mode 100755
index 0000000000..d3722503ab
--- /dev/null
+++ b/tests/btrfs/300
@@ -0,0 +1,52 @@
+#!/bin/bash
+# SPDX-License-Identifier: GPL-2.0
+# Copyright (c) 2023 Meta Platforms, Inc. All Rights Reserved.
+#
+# FS QA Test 300
+#
+# Validate that snapshots taken while in a remapped namespace preserve
+# the permissions of the user.
+#
+. ./common/preamble
+
+_begin_fstest auto quick subvol snapshot
+_register_cleanup "cleanup"
+
+_supported_fs btrfs
+_fixed_by_kernel_commit xxxxxxxxxxxx \
+ "btrfs: copy dir permission and time when creating a stub subvolume"
+
+_require_test
+_require_user
+_require_group
+_require_unix_perm_checking
+_require_unshare
+
+test_dir="${TEST_DIR}/${seq}"
+cleanup() {
+ rm -rf $test_dir
+ cd /
+ rm -rf $tmp.*
+}
+
+rm -rf $test_dir
+mkdir $test_dir
+chown $qa_user:$qa_group $test_dir
+
+# _user_do executes each command as $qa_user in its own subshell. unshare
+# sets the namespace for the running shell. The test must run in one user
+# subshell to preserve the namespace over multiple commands.
+_user_do "
+cd ${test_dir};
+unshare --user --keep-caps --map-auto --map-root-user;
+$BTRFS_UTIL_PROG subvolume create subvol;
+touch subvol/{1,2,3};
+$BTRFS_UTIL_PROG subvolume create subvol/subsubvol;
+touch subvol/subsubvol/{4,5,6};
+$BTRFS_UTIL_PROG subvolume snapshot subvol snapshot;
+"
+
+find $test_dir/. -printf "%M %u %g ./%P\n"
+
+status=0
+exit
diff --git a/tests/btrfs/300.out b/tests/btrfs/300.out
new file mode 100644
index 0000000000..6e94447e87
--- /dev/null
+++ b/tests/btrfs/300.out
@@ -0,0 +1,18 @@
+QA output created by 300
+Create subvolume './subvol'
+Create subvolume 'subvol/subsubvol'
+Create a snapshot of 'subvol' in './snapshot'
+drwxr-xr-x fsgqa fsgqa ./
+drwxr-xr-x fsgqa fsgqa ./subvol
+-rw-r--r-- fsgqa fsgqa ./subvol/1
+-rw-r--r-- fsgqa fsgqa ./subvol/2
+-rw-r--r-- fsgqa fsgqa ./subvol/3
+drwxr-xr-x fsgqa fsgqa ./subvol/subsubvol
+-rw-r--r-- fsgqa fsgqa ./subvol/subsubvol/4
+-rw-r--r-- fsgqa fsgqa ./subvol/subsubvol/5
+-rw-r--r-- fsgqa fsgqa ./subvol/subsubvol/6
+drwxr-xr-x fsgqa fsgqa ./snapshot
+-rw-r--r-- fsgqa fsgqa ./snapshot/1
+-rw-r--r-- fsgqa fsgqa ./snapshot/2
+-rw-r--r-- fsgqa fsgqa ./snapshot/3
+drwxr-xr-x fsgqa fsgqa ./snapshot/subsubvol
diff --git a/tests/overlay/020 b/tests/overlay/020
index 98a33aec23..9f82da344b 100755
--- a/tests/overlay/020
+++ b/tests/overlay/020
@@ -16,18 +16,13 @@ _begin_fstest auto quick copyup perms
# real QA test starts here
-require_unshare() {
- unshare -f -r "$@" true &>/dev/null || \
- _notrun "unshare $@: not supported"
-}
-
# Modify as appropriate.
_supported_fs overlay
_fixed_by_kernel_commit 3fe6e52f0626 \
"ovl: override creds with the ones from the superblock mounter"
_require_scratch
-require_unshare -m -p -U
+_require_unshare
# Remove all files from previous tests
_scratch_mkfs