Message ID | 20231102081601.154862-1-thaller@redhat.com |
---|---|
State | Accepted, archived |
Delegated to: | Florian Westphal |
Headers | show |
Series | [nft,1/1] tests/shell: fix mount command in "test-wrapper.sh" | expand |
Thomas Haller <thaller@redhat.com> wrote: > With Fedora 39 (util-linux-core-2.39.2-1.fc39), the mount command starts > to fail. It was still working with Fedora 38 (util-linux-core-2.38.1-4.fc38). > > $ unshare -f -p -m --mount-proc -U --map-root-user -n bash -c 'mount -t tmpfs --make-private /var/run && mount' > mount: /run: mount failed: Invalid argument. > > Not sure why this starts to fail. But arguably the command line > arguments were wrong. Fix it, we need a pseudo name for the device. Applied, thanks.
diff --git a/tests/shell/helpers/test-wrapper.sh b/tests/shell/helpers/test-wrapper.sh index 872a0c56ed54..b74c56168768 100755 --- a/tests/shell/helpers/test-wrapper.sh +++ b/tests/shell/helpers/test-wrapper.sh @@ -48,7 +48,7 @@ if [ "$NFT_TEST_HAS_UNSHARED_MOUNT" = y ] ; then # Note that this also hides everything that might reside in /var/run. # That is desirable, as tests should not depend on content there (or if # they do, we need to explicitly handle it as appropriate). - if mount -t tmpfs --make-private "/var/run" ; then + if mount -t tmpfs --make-private tmpfs "/var/run" ; then CLEANUP_UMOUNT_VAR_RUN=y fi mkdir -p /var/run/netns
With Fedora 39 (util-linux-core-2.39.2-1.fc39), the mount command starts to fail. It was still working with Fedora 38 (util-linux-core-2.38.1-4.fc38). $ unshare -f -p -m --mount-proc -U --map-root-user -n bash -c 'mount -t tmpfs --make-private /var/run && mount' mount: /run: mount failed: Invalid argument. Not sure why this starts to fail. But arguably the command line arguments were wrong. Fix it, we need a pseudo name for the device. Fixes: df6f1a3e0803 ("tests/shell: bind mount private /var/run/netns in test container") Signed-off-by: Thomas Haller <thaller@redhat.com> --- tests/shell/helpers/test-wrapper.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)