[LU-10369] running "lfs setstripe -c3 /mnt/lustre/pooldir/newfile" doesn't inherit pool

• Type: Bug
• Resolution: Duplicate
• Priority: Major
• Affects Version/s: Lustre 2.10.1
• Severity: 3
• Assignee: Lai Siyao
• Reporter: Andreas Dilger

Description

Testing various scenarios to reproduce LU-10350, I found that running the following test did not work as expected, namely that the file created in pooldir/ did not inherit the pool on the parent directory:

pool_add $POOL
pool_add_targets $POOL 0 1 1
mkdir $tdir/$tdir
lfs setstripe -p $POOL $DIR/$tdir
lfs setstripe -c 3 $DIR/$tdir/$tfile-3stripe
lfs getstripe -p $DIR/$tdir/$tfile-3stripe

lfs setstripe -S 3M $DIR/$tdir/$tfile-3M
lfs getstripe -p $DIR/$tdir/$tfile-3M

lfs setstripe -E 16M -S 1M -c 1 -E 32G -S 4M -c 4 $DIR/$tdir/pfl
lfs getstripe -p $DIR/$tdir/$tfile-3M

None of the created files have inherited the pool from the parent directory. Only if a file is created with "touch" does that work. I thought this was fixed in 2.9 with the PFL code, but it doesn't seem to work.

Also, I notice that sanity.sh and test-framework.sh are using a different set of helper functions for creating pools and adding OSTs (pool_add(), pool_add_targets()) than the ost-pools.sh script (create_pool_fail(), add_pool()). It would be good to remove the duplicate functions and only use one set of pool helpers in all tests.

Issue Links

• is related to: LU-8264 lfs setstripe without -p pool_name doesn't inherit pool from parent/ROOT directory
• is related to: LU-7813 default pool not inherited when specifying any layout parameter

Comments

[Andreas Dilger]

Closing this as a duplicate of LU-8264, since that has a patch.

[Andreas Dilger]

It looks like http://review.whamcloud.com/20849 is supposed to fix this, but it was never landed.

[Andreas Dilger]

Is the problem here that clients/lfs are using LOV_MAGIC_V1 layouts by default, and there is no room for the pool to be inherited? If that is the case, then we could change lfs to use LOV_MAGIC_V3 layouts by default, since that has been supported forever (since 1.6?). Still, it would also be good for old clients/utilities to also inherit the pool properly on the MDS.

[James Nunez (Inactive)]

This ticket is related to and maybe duplicates LU-8264