Whamcloud - gitweb
LU-12399 tests: avoid 'pdsh localhost' in sanity test_420 76/35176/2
authorSebastien Buisson <sbuisson@ddn.com>
Tue, 11 Jun 2019 09:50:01 +0000 (11:50 +0200)
committerOleg Drokin <green@whamcloud.com>
Sun, 16 Jun 2019 03:31:17 +0000 (03:31 +0000)
sanity test_420 needs a clean env to execute openfile, ie not
inherited from root user.
Replace 'pdsh localhost' with simpler 'su - $uname -c' alternative
to achieve this.

Test-Parameters: trivial envdefinitions=ONLY=420 testlist=sanity
Signed-off-by: Sebastien Buisson <sbuisson@ddn.com>
Change-Id: Ifeba7fc1eba86d74a64cca187e286adb23147e2e
Reviewed-on: https://review.whamcloud.com/35176
Tested-by: Jenkins
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Wang Shilong <wshilong@ddn.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
lustre/tests/sanity.sh

index e21d948..579899f 100644 (file)
@@ -20317,7 +20317,7 @@ test_420()
        [ $dirperms == "drwxrwsrwt" ] ||
                error "incorrect perms on $dir/testdir"
 
-       $PDSH ${uname}@localhost "PATH=$LUSTRE/tests:\$PATH; \
+       su - $uname -c "PATH=$LUSTRE/tests:\$PATH; \
                openfile -f O_RDONLY:O_CREAT -m 02755 $dir/testdir/testfile"
        ls -n $dir/testdir/testfile
        local fileperms=$(ls -n $dir/testdir/testfile | awk '{print $1}')