[LU-9486] sanity test 27D failed with 'llapi_layout_test failed'

• Type: Technical task
• Resolution: Fixed
• Priority: Major
• Fix Version/s: Lustre 2.10.0
• Affects Version/s: Lustre 2.10.0
• Labels: pfl
• Assignee: Zhenyu Xu
• Reporter: James Nunez (Inactive)

Description

sanity test_27D fails when a default composite layout is set for the Lustre mount point with

sanity test_27D: @@@@@@ FAIL: llapi_layout_test failed

Looking at the test_log, we can see that sub tests 16 and 20 failed

test 15: Can't change striping attributes of existing file . pass
llapi_layout_test: llapi_layout_test.c:712: test16: assertion 'fcount == dcount' failed: 1 != 1152921504606846979
 test 16: Default stripe attributes are applied as expected . fail (exit status 1)
 test 17: LLAPI_LAYOUT_WIDE is honored ...................... pass
 test 18: Setting pool with fsname.pool notation ............ pass
 test 19: Maximum length pool name is NULL-terminated ....... pass
llapi_layout_test: llapi_layout_test.c:891: test20: assertion 'fcount == dcount' failed: 1 != 1152921504606846979
 test 20: LLAPI_LAYOUT_DEFAULT is honored ................... fail (exit status 1)

Looking at llapi_layout_test.c, it looks like the stripe count for the default layout does not match the count for the file layout

 692         rc = llapi_layout_stripe_count_get(deflayout, &dcount);
 693         ASSERTF(rc == 0, "errno = %d", errno);
 694 
 695         /* First, with a default struct llapi_layout */
 696         filelayout = llapi_layout_alloc();
 697         ASSERTF(filelayout != NULL, "errno = %d", errno);
 698 
 699         fd = llapi_layout_file_create(path, 0, 0640, filelayout);
 700         ASSERTF(fd >= 0, "errno = %d", errno);
 701 
 702         rc = close(fd);
 703         ASSERTF(rc == 0, "errno = %d", errno);
 704 
 705         llapi_layout_free(filelayout);
 706 
 707         filelayout = llapi_layout_get_by_path(path, 0);
 708         ASSERTF(filelayout != NULL, "errno = %d", errno);
 709 
 710         rc = llapi_layout_stripe_count_get(filelayout, &fcount);
 711         ASSERTF(rc == 0, "errno = %d", errno);
 712         ASSERTF(fcount == dcount, "%"PRIu64" != %"PRIu64, fcount, dcount);

Log for recent failures are at
https://testing.hpdd.intel.com/test_sets/2e962ed8-3534-11e7-814a-5254006e85c2
https://testing.hpdd.intel.com/test_sets/f5d085aa-357c-11e7-b0a8-5254006e85c2

Issue Links

• is related to: LU-6156 sanity test 27D failure: llapi_layout_test failed
• is related to: LU-9324 sanity-pfl test 10 needs to reset the file system default layout

Comments

[Peter Jones]

Landed for 2.10

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/27057/
Subject: LU-9486 llapi-test: sanity test_27D failure
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: f20aa2305067504185684946195e9e1bb1a68a49

[Gerrit Updater]

Bobi Jam ([e-mail]) uploaded a new patch: https://review.whamcloud.com/27057
Subject: LU-9486 llapi-test: sanity test_27D failure
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: f9e054dd196b6b659281ed448b798e03ac56c429

[Zhenyu Xu]

1152921504606846979 == 0x1000000000000003 == LLAPI_LAYOUT_WIDE
/**

• When specified or returned as the value for stripe count, all
• available OSTs will be used.
  */
  #define LLAPI_LAYOUT_WIDE (LLAPI_LAYOUT_INVALID + 2)

When the default mount point layout contains -1 stripe_count, this value is returned as it's stripe_count.