DAOS-18640 test: per_server_fault_domain.yaml - Add space_rb:5 to poo…#17638
Open
shimizukko wants to merge 5 commits intomasterfrom
Open
DAOS-18640 test: per_server_fault_domain.yaml - Add space_rb:5 to poo…#17638shimizukko wants to merge 5 commits intomasterfrom
shimizukko wants to merge 5 commits intomasterfrom
Conversation
…l prop test_rf2_unclean is failing due to the rebuild timeout after stopping the ranks. This may be due to not enough space for rebuild. The pool size is 200G and 20GiB data are written with IOR. However, the reserved space for rebuild (space_rb) is 0, which may not be enough to rebuild 20GiB data. Increase space_rb to 8% to allow rebuild to have some space to work. Skip-unit-tests: true Skip-fault-injection-test: true Skip-func-hw-test-medium: false Test-tag: test_rf2_unclean Signed-off-by: Makito Kano <makito.kano@hpe.com>
|
Ticket title is 'container/per_server_fault_domain.py:PerServerFaultDomainTest.test_rf2_unclean - wait for rebuild test timeout - DER_NOSPACE' |
Skip-unit-tests: true Skip-fault-injection-test: true Skip-func-hw-test-large: false Test-tag: test_rf2_unclean Test-repeat: 3 Signed-off-by: Makito Kano <makito.kano@hpe.com>
Skip-unit-tests: true Skip-fault-injection-test: true Skip-func-hw-test-large: false Test-tag: test_rf2_unclean Test-repeat: 3 Signed-off-by: Makito Kano <makito.kano@hpe.com>
daltonbohning
approved these changes
Mar 4, 2026
| ranks_to_stop_str = str(rank) | ||
| else: | ||
| ranks_to_stop_str += "," + str(rank) | ||
| ranks_to_stop_str = ",".join(str(rank) for rank in ranks_to_stop) |
Contributor
There was a problem hiding this comment.
It's a matter of preference but FYI the map function is useful for things like this
Suggested change
| ranks_to_stop_str = ",".join(str(rank) for rank in ranks_to_stop) | |
| ranks_to_stop_str = ",".join(map(str, ranks_to_stop)) |
daltonbohning
previously approved these changes
Mar 4, 2026
phender
requested changes
Mar 4, 2026
Skip-unit-tests: true Skip-fault-injection-test: true Skip-func-hw-test-large: false Test-tag: test_rf2_unclean Test-repeat: 3
Use map to create comma-separated list of ranks. Skip-unit-tests: true Skip-fault-injection-test: true Skip-func-hw-test-large: false Test-tag: test_rf2_unclean Test-repeat: 3 Signed-off-by: Makito Kano <makito.kano@hpe.com>
shimizukko
changed the title
daltonbohning
approved these changes
Mar 5, 2026
phender
approved these changes
Mar 5, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…l prop
test_rf2_unclean is failing due to the rebuild timeout after stopping the ranks. This may be due to not enough space for rebuild.
The pool size is 200G and 20GiB data are written with IOR. However, the reserved space for rebuild (space_rb) is 0. Increase space_rb to 5% to allow rebuild to have some space to work.
Skip-unit-tests: true
Skip-fault-injection-test: true
Skip-func-hw-test-medium: false
Test-tag: test_rf2_unclean
Steps for the author:
After all prior steps are complete: