[LU-3313] Interop 2.1.5<->2.4 failure on test suite sanityn test_51b: file size is 0, should be 1024

• Status: Resolved
• Project: Lustre
• Affects Version/s: Lustre 2.4.0
• Fix Version/s: Lustre 2.4.1, Lustre 2.5.0
• Type: Bug
• Priority: Minor
• Reporter: Maloo
• Assignee: James Nunez (Inactive)
• Resolution: Fixed

Environment

server: 2.1.5
client: lustre-master build #1481

• Severity: 3

Description

This issue was created by maloo for sarah <[e-mail]>

This issue relates to the following test suite run: http://maloo.whamcloud.com/test_sets/a75bb926-b88f-11e2-891d-52540035b04c.

The sub-test test_51b failed with the following error:

file size is 0, should be 1024

client console:

05:54:15:Lustre: DEBUG MARKER: == sanityn test 51b: layout lock: glimpse should be able to restart if layout changed == 05:54:11 (1368017651)
05:54:15:LustreError: 21920:0:(fail.c:133:__cfs_fail_timeout_set()) cfs_fail_timeout id 1404 sleeping for 2000ms
05:54:15:LustreError: 21920:0:(fail.c:137:__cfs_fail_timeout_set()) cfs_fail_timeout id 1404 awake
05:54:15:Lustre: DEBUG MARKER: /usr/sbin/lctl mark  sanityn test_51b: @@@@@@ FAIL: file size is 0, should be 1024 
05:54:15:Lustre: DEBUG MARKER: sanityn test_51b: @@@@@@ FAIL: file size is 0, should be 1024

Comments

[Jodi Levi (Inactive)]

James to discuss with Jinshan to see where this test should be skipped.

[Andreas Dilger]

Please skip test if server does not have layout lock. This can be checked by the MDS version < 2.3.60 or so ("git annotate lustre/tests/sanity.sh" to see in what commit this test was added, then "git describe --contains

{hash}

" to get the tag that includes this patch.

[James Nunez (Inactive)]

Thanks for the commands, Andreas.

# git annotate tests/sanityn.sh
...
5517eab0        (Jinshan Xiong  2012-12-06 09:34:15 -0800       2303)test_51b() {
...
# git describe --contains 5517eab0 
2.3.59~20

So, layout lock for the server was added in 2.3.59. So, we'll skip the test for MDS version < 2.3.59.

[James Nunez (Inactive)]

A proposed patch is at:

http://review.whamcloud.com/#change,6340

[James Nunez (Inactive)]

Patch landed to master.

[Jian Yu]

http://review.whamcloud.com/#change,6340

The patch also needs to be cherry-picked to Lustre b2_4 branch to resolve interop failure with old servers.

[James Nunez (Inactive)]

A patch for b2_4 is at

http://review.whamcloud.com/#/c/7051/