[LU-14480] Setting specific OST's under pool failed with error message "invalid argument"

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.15.0
• Severity: 3
• Assignee: Vitaly Fertman
• Reporter: Vitaly Fertman

Description

the Error appears if the specific OST is not in the pool. at the same time, only the first OST is checked.

the regular setstripe and overstripe's -C option works fine and the same way - they allocate OSTs within the given pool starting with the -i first OST. if the starting index is out of the pool, it is always enough to check only the first ost index in lod_qos_parse_config() ->lod_check_index_in_pool()

however the -o option gives a new opportunity to list all the OSTs we want to place stripes to. and checking only the 1st index is not enough - as the result we may get an error or succeed depending on the 1st index only.

[root@vm71 wc-main]# mkdir /mnt/lustre/d1
[root@vm71 wc-main]# ./lustre/utils/lctl pool_new lustre.lala
./lustre/utils/lctl pool_new lustre.kaka
Pool lustre.lala created
[root@vm71 wc-main]# ./lustre/utils/lctl pool_new lustre.kaka
Pool lustre.kaka created
[root@vm71 wc-main]# ./lustre/utils/lctl pool_add lustre.lala lustre-OST[0]
./lustre/utils/lctl pool_add lustre.kaka lustre-OST[1]
OST lustre-OST0000_UUID added to pool lustre.lala
[root@vm71 wc-main]# ./lustre/utils/lctl pool_add lustre.kaka lustre-OST[1]
OST lustre-OST0001_UUID added to pool lustre.kaka
[root@vm71 wc-main]# ./lustre/utils/lfs setstripe --pool lustre.lala /mnt/lustre/d1/
[root@vm71 wc-main]# ./lustre/utils/lfs setstripe -o 1,0 /mnt/lustre/d1/file2
lt-lfs setstripe: setstripe error for '/mnt/lustre/d1/file2': Invalid argument
[root@vm71 wc-main]# ./lustre/utils/lfs setstripe -o 0,1 /mnt/lustre/d1/file2

the given OST list should have a priority over the pool.

Issue Links

• is related to: LU-15658 Interop sanity-flr test_0b test_0c test_0e test_0f: verify pool failed != flash
• is related to: LU-15707 Unable to create file without a pool with a pool define on the parent or on the fs root.
• is related to: LU-17703 some pool inheritance fixes
• is related to: LU-9392 lfs migrate -o and lfs setstripe -o should pick OST from ost_list

Comments

[Gerrit Updater]

"Etienne AUJAMES <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/46362
Subject: LU-14480 pool: wrong usage with ost list
Project: fs/lustre-release
Branch: b2_12
Current Patch Set: 1
Commit: 1a015901ec9a699fb13c502b01b04e9071ee029c

[Peter Jones]

Landed for 2.15

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/41815/
Subject: LU-14480 pool: wrong usage with ost list
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: b384ea39e593cda1ac4d6fb8b955d0c7d1a1f67b

[Gerrit Updater]

Vitaly Fertman ([e-mail]) uploaded a new patch: https://review.whamcloud.com/41815
Subject: LU-14480 pool: wrong usage with ost list
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: d50ae8bd096f7fc2fb88f9a99ace0e03665b095d