[LU-12399] sanity test_420: FAIL: incorrect perms on /mnt/lustre/d420.sanity/testdir/testfile

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.13.0, Lustre 2.12.3
• 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
• Severity: 3
• Assignee: Sebastien Buisson
• Reporter: Jian Yu

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

Issue Links

• is related to: LU-12269 Support RHEL 8.0

Comments

[Gerrit Updater]

Oleg Drokin ([e-mail]) 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

[Gerrit Updater]

Jian Yu ([e-mail]) 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

[Peter Jones]

Landed for 2.13

[Gerrit Updater]

Oleg Drokin ([e-mail]) 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

[Gerrit Updater]

Sebastien Buisson ([e-mail]) 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

[Sebastien Buisson]

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

[Sebastien Buisson]

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.

[Wang Shilong (Inactive)]

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

[Jian Yu]

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".

[Sebastien Buisson]

Oops, sorry

 

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