On Wed, Feb 19, 2025 at 11:50:03AM -0500, Jeff Layton wrote: > I had been seeing some failures in generic/126 when running on NFS under > kdevops. > > The problem turned out to be that kdevops makes the root of the NFS > export mode 01777 by default (i.e. with the sticky bit set). This causes > some of these permission tests to fail because the open() is called with > the O_CREAT bit set. > > Make the outcome of this test not depend on the permissions on > $TEST_DIR. Create a new directory to do these tests in, and clean it up > when done. > > Signed-off-by: Jeff Layton <jlayton@xxxxxxxxxx> > --- > tests/generic/126 | 11 +++++++++-- > 1 file changed, 9 insertions(+), 2 deletions(-) > > diff --git a/tests/generic/126 b/tests/generic/126 > index 142d2bb97db706bafd2152856ef0993c5c273441..fe0d6db1b6bd85d4942333fdf04d23a63c4052ec 100755 > --- a/tests/generic/126 > +++ b/tests/generic/126 > @@ -23,7 +23,13 @@ _require_chown > > QA_FS_PERMS=$here/src/fs_perms > > -cd $TEST_DIR > +testdir="$TEST_DIR/generic-126.$$" > + > +mkdir $testdir > +chown 0:0 $testdir > +chmod 0755 $testdir > +cd $testdir Comment explaining why this weird setup step is done, please! > + > cp $here/src/testx ./testx.file > > # file_perm owner_uid owner_gid tester_uid tester_gid perm_to_test expected_result pass=1 > @@ -46,7 +52,8 @@ $QA_FS_PERMS 200 99 99 200 99 w 1 > $QA_FS_PERMS 040 99 99 99 500 r 1 > $QA_FS_PERMS 400 99 99 200 99 r 1 > > -rm -f ./testx.file > +cd / > +rm -rf $testdir belongs in _cleanup(), not here. -Dave. -- Dave Chinner david@xxxxxxxxxxxxx
