Details
-
Improvement
-
Resolution: Unresolved
-
Minor
-
None
-
None
-
9223372036854775807
Description
The test-framework.sh helper functions mdts_nodes(), osts_nodes(), agts_nodes(), nodes_list(), all_mdts_nodes(), all_osts_nodes(), all_server_nodes() and all_nodes() are mostly used together with "comma_list()" to convert the space-separated node list into a comma-separated node list suitable for use by pdsh. There are 261 uses of these functions, of which only 55 do not immediately call comma_list(),
It makes sense to change these helper functions to call comma_list() internally to always return the comma-separated list of nodes, and fix the few places that need a space-separated list to use "| tr ',' ' '" or "${VAR//,/ }" or similar.
It shouldn't be harmful for "comma_list()" to take an already comma-separated list as input, so there isn't a requirement for a single patch to remove the use of comma_list() from all tests, only those few places that require a space-separated list.
Attachments
Activity
"Andreas Dilger <adilger@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/58025
Subject: LU-12597 tests: remove comma_list for MDTs part 7
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 7f0b616225d0e6b409c06c1b8cb9657dd996cc13
"Andreas Dilger <adilger@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/58024
Subject: LU-12597 tests: remove comma_list for MDTs part 6
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: e601cff45a71c492a7b023d89e3a190d1ba7d042
"Andreas Dilger <adilger@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/58023
Subject: LU-12597 tests: remove comma_list for MDTs part 5
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: e418f57cda7dc2397a6e5885c51870c802078ef6
"Andreas Dilger <adilger@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/58022
Subject: LU-12597 tests: remove comma_list for MDTs part 4
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 0f8ee74aa09cf47a52d200ed43b9ee6c26958bd6
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/57326/
Subject: LU-12597 tests: remove comma_list for MDTs part 3
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: ceee271cddc3a911a54415d7e5a125c25e6ab35b
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/57325/
Subject: LU-12597 tests: remove comma_list for MDTs part 2
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 178e5b2c399a0b8adf3642eae1be705103cf5383
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/57183/
Subject: LU-12597 tests: remove comma_list for MDTs part 1
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: f2642a2947d1fc109c70aadbc86c6510df9b8a34
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/57194/
Subject: LU-12597 tests: return comma-separated mdts_nodes()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: a2d7e81c01e913d7d4401ab317b2b71652753576
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/57132/
Subject: LU-12597 tests: remove comma_list for OSTs part 4
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: a173c24d5aba7b3a5b5d53d29f7b3a636e469f91
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/56909/
Subject: LU-12597 tests: remove comma_list for OSTs part 3
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 87beee4a74076c2c7c3a45ea0c6d42446be63c4f
"Andreas Dilger <adilger@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/58026
Subject: LU-12597 tests: remove comma_list for MDTs part 8
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: e31ab269becffa729a4c245536f2d42e8655b6ca