[LU-7471] In Lustre sanity HSM tests make_large_for_*() functions shadow free space failure

• Status: Resolved
• Project: Lustre
• Fix Version/s: Lustre 2.10.0
• Type: Bug
• Priority: Minor
• Reporter: Aditya Pandit (Inactive)
• Assignee: Emoly Liu
• Resolution: Fixed
• Labels: patch
• Severity: 3

Description

1. Prepare Lustre with less than 100MB of free space.
2. Run test 200 of HSM sanity check in prepared Lustre.

Issue reproduced if test fails (typically due to errors in awk script called later in the test function).

Issue fixed if test ends with 'SKIP' outcome and message "not enough free space".

Comments

[Gerrit Updater]

aditya.pandit ([e-mail]) uploaded a new patch: http://review.whamcloud.com/17346
Subject: LU-7471 tests: Modified make_large_for fn to return correctly
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 99425984fc497cf1a6dc894f8749f6166b0a09b1

[Ashish Purkar (Inactive)]

Better fix is already in progress(review) at http://review.whamcloud.com/#/c/20442/ in LU-8203.

[Emoly Liu]

[e-mail], as Ashish said, do you think the fix http://review.whamcloud.com/#/c/20442/ is better than http://review.whamcloud.com/17346 for this issue? Since 20442 has landed, can we abandon 17345?

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/17346/
Subject: LU-7471 tests: Modified make_custom_file_for_progress fn
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 2dbb4d1ae3bcda5c733c512df141837289e03b7a

[Peter Jones]

Is it ok that the second fix landed too or will this have undone the preferred first fix?

[Ashish Purkar (Inactive)]

Both the fixes are needed and fix #17346 was updated on Jun 22 after the fix #20442 was landed to master.