[LU-12399] sanity test_420: FAIL: incorrect perms on /mnt/lustre/d420.sanity/testdir/testfile Created: 06/Jun/19  Updated: 14/Dec/21  Resolved: 16/Jun/19

Status: Resolved
Project: Lustre
Component/s: None
Affects Version/s: None
Fix Version/s: Lustre 2.13.0, Lustre 2.12.3

Type: Bug Priority: Minor
Reporter: Jian Yu Assignee: Sebastien Buisson
Resolution: Fixed Votes: 0
Labels: None
Environment:

Lustre build: https://build.whamcloud.com/job/lustre-master/3904/ (tag 2.12.54)
Lustre client distro: RHEL 8.0
Lustre server distro: RHEL 7.6


Issue Links:
Related
is related to LU-12269 Support RHEL 8.0 Resolved
Severity: 3
Rank (Obsolete): 9223372036854775807

 Description   

sanity test 420 failed as follows:

== sanity test 420: clear SGID bit on non-directories for non-members ================================ 11:24:44 (1559845484)
drwxrwsrwt 2 0 0 4096 Jun  6 11:24 /mnt/lustre/d420.sanity/testdir
localhost: Permission denied.
ls: cannot access '/mnt/lustre/d420.sanity/testdir/testfile': No such file or directory
ls: cannot access '/mnt/lustre/d420.sanity/testdir/testfile': No such file or directory
/usr/lib64/lustre/tests/sanity.sh: line 20215: [: ==: unary operator expected
 sanity test_420: @@@@@@ FAIL: incorrect perms on /mnt/lustre/d420.sanity/testdir/testfile 

Maloo report: https://testing.whamcloud.com/test_sets/f180e12c-8889-11e9-be83-52540065bddc



 Comments   
Comment by Peter Jones [ 06/Jun/19 ]

Sébastien

Can you please investigate?

Thanks

Peter

Comment by Sebastien Buisson [ 07/Jun/19 ]

It looks like a configuration issue on the RHEL8 client node.

The command that fails is:

$PDSH ${uname}@localhost "openfile ..."

On RHEL7.6 client nodes in Jenkins, such pdsh command works.

Comment by Sebastien Buisson [ 07/Jun/19 ]

I opened DCO-8348 to report this pdsh problem.

Comment by Minh Diep [ 07/Jun/19 ]

keep in mind that the report on this ticket is from manual testing uploaded, not from autotest

Comment by Sebastien Buisson [ 07/Jun/19 ]

Oops, sorry

 

So my recommendation is to allow 'pdsh localhost' on client node running this test.

Comment by Jian Yu [ 07/Jun/19 ]

Since the command needs to be run on local client, it seems we can just use $RUNAS instead of $PDSH, so that user do not need to configure "$PDSH ${uname}@localhost".

Comment by Wang Shilong (Inactive) [ 11/Jun/19 ]

I agreed with Jian, we hit the failure too locally testing, just running $RUNAS might be better.

Comment by Sebastien Buisson [ 11/Jun/19 ]

No, $RUNAS is not a substitute.
As explained in https://review.whamcloud.com/34809, I tried with runas, but it does not work, because it only substitutes the UID and GID.

However, if you are able to find an alternative to $PDSH that makes sanity test_420 pass, I would be glad to use it.

Comment by Sebastien Buisson [ 11/Jun/19 ]

It turns out that a simple alternative is to use 'su - $uname -c'. I will push a patch with this fix.

Comment by Gerrit Updater [ 11/Jun/19 ]

Sebastien Buisson (sbuisson@ddn.com) uploaded a new patch: https://review.whamcloud.com/35176
Subject: LU-12399 tests: avoid 'pdsh localhost' in sanity test_420
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 30f8f143785a3ccf44af17db3c15cc23b9d7d9e5

Comment by Gerrit Updater [ 16/Jun/19 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35176/
Subject: LU-12399 tests: avoid 'pdsh localhost' in sanity test_420
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 1476ac047b449886a0c382b840a7b09dc0cec7eb

Comment by Peter Jones [ 16/Jun/19 ]

Landed for 2.13

Comment by Gerrit Updater [ 17/Jun/19 ]

Jian Yu (yujian@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/35250
Subject: LU-12399 tests: avoid 'pdsh localhost' in sanity test_420
Project: fs/lustre-release
Branch: b2_12
Current Patch Set: 1
Commit: 1910e52fa8932eeae093edb20b81667f144884b6

Comment by Gerrit Updater [ 03/Jul/19 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35250/
Subject: LU-12399 tests: avoid 'pdsh localhost' in sanity test_420
Project: fs/lustre-release
Branch: b2_12
Current Patch Set:
Commit: 4df2ac9ddc717b435773ddf0bbe5c89ac0c9fda0

Generated at Sat Feb 10 02:52:14 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.