Skip to content

Conversation

@shimizukko
Copy link
Contributor

Update get_vos_path to get_vos_paths so that it returns all vos paths such as /mnt/daos0, /mnt/daos1, and so on.

Update the tests calling get_vos_path.

Skip-unit-tests: true
Skip-fault-injection-test: true
Skip-func-hw-test-medium: false
Test-tag: test_check_start_reset test_check_start_failout test_orphan_pool_trust_ms test_lost_majority_ps_replicas test_lost_all_rdb test_dangling_pool_map test_dangling_rank_entry test_orphan_container DdbTest

Steps for the author:

  • Commit message follows the guidelines.
  • Appropriate Features or Test-tag pragmas were used.
  • Appropriate Functional Test Stages were run.
  • At least two positive code reviews including at least one code owner from each category referenced in the PR.
  • Testing is complete. If necessary, forced-landing label added and a reason added in a comment.

After all prior steps are complete:

  • Gatekeeper requested (daos-gatekeeper added as a reviewer).

Update get_vos_path to get_vos_paths so that it returns
all vos paths such as /mnt/daos0, /mnt/daos1, and so on.

Update the tests calling get_vos_path.

Skip-unit-tests: true
Skip-fault-injection-test: true
Skip-func-hw-test-medium: false
Test-tag: test_check_start_reset test_check_start_failout test_orphan_pool_trust_ms test_lost_majority_ps_replicas test_lost_all_rdb test_dangling_pool_map test_dangling_rank_entry test_orphan_container DdbTest
Signed-off-by: Makito Kano <[email protected]>
@github-actions
Copy link

github-actions bot commented Jan 9, 2026

Ticket title is 'CR Test Fix - util/server_utils.py Update get_vos_path'
Status is 'Open'
Labels: 'catastrophic_recovery'
https://daosio.atlassian.net/browse/DAOS-18403

@shimizukko shimizukko requested review from dinghwah and phender January 9, 2026 11:36
@shimizukko shimizukko marked this pull request as ready for review January 9, 2026 11:37
@shimizukko shimizukko requested review from a team as code owners January 9, 2026 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants