[LU-7803] sanity test 78 failures in interop

• Type: Bug
• Resolution: Fixed
• Priority: Major
• Fix Version/s: Lustre 2.9.0
• Severity: 3
• Assignee: Nathaniel Clark
• Reporter: Oleg Drokin

Description

Addition of LU-2049 patch http://review.whamcloud.com/7793 seems to be breaking interop in particular test 78 of sanity.sh so I filing this ticket just in case.

I am testing with upstream client, but 2.5 and 2.7 at the very least are likely affected, possibly even 2.8, so we need to double check.

With the patch in place on the server, client running test 78 fails on the 5th iteration like this:
directIO rdwr round 5 of 5
directio on /mnt/lustre/f78.sanity for 130x1048576 bytes
Write error Success (rc = 83886080, len = 136314880)
sanity test_78: @@@@@@ FAIL: rdwr failed

This suggests the write was short due to overzealous limitation of free space on the server or something else.

I confirmed that removing that patch fixes the failure.

additional issue - test 78 does not cleanup the big file on error leading to a chain of out of space failures later on in the sanity.sh at least in my setup

This is on ldiskfs

Issue Links

• is related to: LU-2049 add support for OBD_CONNECT_GRANT_PARAM

Comments

[Peter Jones]

Landed for 2.9

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch http://review.whamcloud.com/21808/
Subject: LU-7803 tests: Cleanup after sanity/78
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 76c696d545677bab4ce9641b9eff338fe99f6396

[Oleg Drokin]

hm, this won't fix failure in the patchless upstream client, though, and it typically uses master scripts.
When run manually it might be ignored, but it is also run auomaticlaly at times, so I wonder how to better approach this for that case?

I guess not much could be done considering we don't use master scripts at all times there too...

[Gerrit Updater]

Nathaniel Clark ([e-mail]) uploaded a new patch: http://review.whamcloud.com/21808
Subject: LU-7803 tests: Cleanup after sanity/78
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 32cdae7e13fbe23f0a8caf165ddb1ac7331ddffa