[LU-8264] lfs setstripe without -p pool_name doesn't inherit pool from parent/ROOT directory

• Type: Bug
• Resolution: Fixed
• Priority: Critical
• Fix Version/s: Lustre 2.11.0, Lustre 2.10.4
• Affects Version/s: Lustre 2.9.0, Lustre 2.10.1, Lustre 2.10.4
• Labels: patch
• Severity: 3
• Assignee: Hongchao Zhang
• Reporter: Rahul Deshmukh (Inactive)

Description

Actually exactly sure of if this is bug, as a doubt filing this bug. Please adviced. Here is use case or more details about it.

Say I have 4 osts on my setup and created pool with 2 osts assign to it

lctl pool_new lustre.pool ;
lctl pool_add lustre.pool lustre-OST0000 lustre-OST0001 ; 
mkdir /mnt/lustre/test_pool
lfs setstripe -c -1 -p lustre.pool /mnt/lustre/test_pool 

Now I can create file in that by following two ways

 case1: lfs setstripe -c 3 -p lustre.pool /mnt/lustre/test_pool/test1
 case2: lfs setstripe -c 3  /mnt/lustre/test_pool/test2 

Here in case 1 we trim the strip count to 2 as pool has 2 osts
But in case 2 we allow stripe count 3
So question is case2 is expected behaviour or bug (as test_pool is assigned to pool) ?

Following text taken from admin guide.

Several lfs commands can be run with OST pools. Use the lfs setstripe command to associate a directory with an OST pool. This causes all new regular files and directories in the directory to be created in the pool. The lfs command can be used to list pools in a file system and OSTs in a named pool.

Issue Links

• duplicates: LU-7660 FS default striping settings only honored on MDT 0
• is related to: LU-10369 running "lfs setstripe -c3 /mnt/lustre/pooldir/newfile" doesn't inherit pool
• is related to: LU-7813 default pool not inherited when specifying any layout parameter
• is related to: LU-10350 ost-pools test 1n fails with 'failed to write to /mnt/lustre/d1n.ost-pools/file: 1'

Comments

[Andreas Dilger]

Patch http://review.whamcloud.com/20859 "LU-8264 tests: setstripe striping parameters not inherited" looks like it has the test case for this change, it just needs to be updated and landed.

[Lai Siyao]

sure, I'll add a specific test case later.

[Andreas Dilger]

Since 20849 is landed for 2.9, can you please convert "IMO" to a test case to verify this is working correctly so this bug can be closed.

[Lai Siyao]

This issue is that when striping is specified, but pool not, the inherited pool (from parent or filesystem default) should be used. However current code always ignore inherited pool.

IMO http://review.whamcloud.com/20849 can fix this issue.

[Andreas Dilger]

Lai, can you please verify that this issue is fixed with your patch http://review.whamcloud.com/19041 "LU-7660 dne: support fs default stripe" and then close this ticket if it is. Otherwise, please add a comment explaining what still needs to be done.

[Rahul Deshmukh (Inactive)]

Rahul - I just made a few changes to ost-pools.sh to catch this kind of error. I'll rebase my patch when yours lands to master and make any necessary changes.

Ok, Sure.

[James Nunez (Inactive)]

Rahul - I just made a few changes to ost-pools.sh to catch this kind of error. I'll rebase my patch when yours lands to master and make any necessary changes.

Some of the changes I made to the ost-pools test script were based on Andreas' abandoned patch http://review.whamcloud.com/#/c/18887/

[Gerrit Updater]

James Nunez ([e-mail]) uploaded a new patch: http://review.whamcloud.com/20859
Subject: LU-8264 tests: setstripe striping parameters not inherited
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 9abd9545d25a8a27fb6542c3077453a14fe35184

[Gerrit Updater]

Rahul Deshmukh ([e-mail]) uploaded a new patch: http://review.whamcloud.com/20849
Subject: LU-8264 lod: lfs setstripe fix for pool.
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 81089bfb4f3564cb2f7ba40d5c39598da2403768

[Rahul Deshmukh (Inactive)]

> This should be fixed by the patch http://review.whamcloud.com/19041 "LU-7660 dne: support fs default stripe" which would be very
> useful if you could test and verify that it is fixing this problem.

I have checked with above mention patch but it did not solve the problem. Let me cook the patch and push it.

[James Nunez (Inactive)]

Seems like ost-pools.sh test 16 "Inheritance of pool properties" falls short. I'll work on improving this test.