[LU-9741] sanity test 27w failed with “/usr/bin/lfs getstripe -d /mnt/lustre/d27w.sanity failed”

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.11.0, Lustre 2.10.2
• Affects Version/s: Lustre 2.10.0
• Labels: pfl test
• Severity: 3
• Assignee: James Nunez (Inactive)
• Reporter: James Nunez (Inactive)

Description

Running the sanity test suite with a composite file layout set on the mount point results in sanity test_27w failing with

sanity test_27w: @@@@@@ FAIL: /usr/bin/lfs getstripe -d /mnt/lustre/d27w.sanity failed

Test 27w gets stripe information for a directory and assumes that there is only one “stripe_count” entry in the output;

1715 test_27w() { # bug 10997
1716         test_mkdir -p $DIR/$tdir || error "mkdir failed"
1717         $SETSTRIPE -S 65536 $DIR/$tdir/f0 || error "setstripe failed"
1718         [ $($GETSTRIPE -S $DIR/$tdir/f0) -ne 65536 ] &&
1719                 error "stripe size $size != 65536" || true
1720         [ $($GETSTRIPE -d $DIR/$tdir | grep -c "stripe_count") -ne 1 ] &&
1721                 error "$GETSTRIPE -d $DIR/$tdir failed" || true
1722 }

With PFL, directories can have more than one component. For example, if we use ‘lfs setstripe’ to set the component count on a directory greater than one and get the stripe information with ‘lfs getstripe’, we see there is more than one “stripe_count” entry in the output

# lfs setstripe -E 2M -S 1M -c 1 -E -1 -c -1 /lustre/scratch
# lfs getstripe /lustre/scratch/
/lustre/scratch/
  lcm_layout_gen:  0
  lcm_entry_count: 2
    lcme_id:             N/A
    lcme_flags:          0
    lcme_extent.e_start: 0
    lcme_extent.e_end:   2097152
      stripe_count:  1       stripe_size:   1048576       stripe_offset: -1
    lcme_id:             N/A
    lcme_flags:          0
    lcme_extent.e_start: 2097152
    lcme_extent.e_end:   EOF
      stripe_count:  -1       stripe_size:   1048576       stripe_offset: -1
# mkdir /lustre/scratch/testdir
# lfs getstripe -d /lustre/scratch/testdir
  lcm_layout_gen:  0
  lcm_entry_count: 2
    lcme_id:             N/A
    lcme_flags:          0
    lcme_extent.e_start: 0
    lcme_extent.e_end:   2097152
      stripe_count:  1       stripe_size:   1048576       stripe_offset: -1
    lcme_id:             N/A
    lcme_flags:          0
    lcme_extent.e_start: 2097152
    lcme_extent.e_end:   EOF
      stripe_count:  -1       stripe_size:   1048576       stripe_offset: -1
# lfs getstripe -d /lustre/scratch/testdir | grep -c stripe_count
2

It looks like this test needs to be modified to not assume that there is only one stripe_count entry for directories.

Logs for one of these failures is at
https://testing.hpdd.intel.com/test_sets/f3414818-5bcb-11e7-a749-5254006e85c2

Comments

[Gerrit Updater]

John L. Hammond ([e-mail]) merged in patch https://review.whamcloud.com/29625/
Subject: LU-9741 test: Correct check of stripe count for directories
Project: fs/lustre-release
Branch: b2_10
Current Patch Set:
Commit: bfe04c04f7eaa81809e42c06f191a266804c8834

[Gerrit Updater]

Minh Diep ([e-mail]) uploaded a new patch: https://review.whamcloud.com/29625
Subject: LU-9741 test: Correct check of stripe count for directories
Project: fs/lustre-release
Branch: b2_10
Current Patch Set: 1
Commit: c867ea4ed6f02455f98d010f64817086e4bfbcfa

[Peter Jones]

Landed for 2.11

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/27980/
Subject: LU-9741 test: Correct check of stripe count for directories
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 1cc354d559d1df90a2eaa886eb0e3956bf385a9e

[Gerrit Updater]

James Nunez ([e-mail]) uploaded a new patch: https://review.whamcloud.com/27980
Subject: LU-9741 test: Correct check of stripe count for directories
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 0fdec8787078127c24bb7b9a6ebadccdae6b1d2b

[James Nunez (Inactive)]

Andreas - From the example in the description, the number of lines with "stripe count" in them in the 'lfs getstripe -d' output for directories with a composite layout is now dynamic; one line with "stripe_count" in it per component.

So, you're suggesting that we want to see one or more lines with "stripe count" in the 'lfs getstripe -d' output. That sounds reasonable. Thank you. I'll upload a patch.

[Andreas Dilger]

This should be as simple as changing -ne 1 to -eq 0.