Skip to content

DAOS-16575 rebuild: set rgt DRS_COMPLETED for unretryable failure#18232

Open
liuxuezhao wants to merge 1 commit into
release/2.8from
lxz/rb_test_refine_28
Open

DAOS-16575 rebuild: set rgt DRS_COMPLETED for unretryable failure#18232
liuxuezhao wants to merge 1 commit into
release/2.8from
lxz/rb_test_refine_28

Conversation

@liuxuezhao
Copy link
Copy Markdown
Contributor

If rebuild failed with unretryable failure, set it as DRS_COMPLETED to avoid following pool query always report rebuild busy such as - Rebuild failing (state=busy, status=-2001)
after this patch, that kind of failed rebuild will report - Rebuild failed (state=done, status=-2001)
This can fix test_rebuild_31/REBUILD31 problem.

For test_rebuild_32/REBUILD32 add DAOS_POOL_TGT_UPDATE_SKIP_RF_CHECK to bypass DAOS_POOL_RF checking when changing pool map.

Fixes: DAOS-16575 DAOS-16576
Test-tag: pr test_rebuild_31 test_rebuild_32

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).

If rebuild failed with unretryable failure, set it as DRS_COMPLETED
to avoid following pool query always report rebuild busy such as -
Rebuild failing (state=busy, status=-2001)
after this patch, that kind of failed rebuild will report -
Rebuild failed (state=done, status=-2001)
This can fix test_rebuild_31/REBUILD31 problem.

For test_rebuild_32/REBUILD32 add DAOS_POOL_TGT_UPDATE_SKIP_RF_CHECK to
bypass DAOS_POOL_RF checking when changing pool map.

Fixes: DAOS-16575 DAOS-16576
Test-tag: pr test_rebuild_31 test_rebuild_32

Signed-off-by: Xuezhao Liu <xuezhao.liu@hpe.com>
@liuxuezhao liuxuezhao requested review from a team as code owners May 12, 2026 15:55
Copy link
Copy Markdown
Contributor

@daltonbohning daltonbohning left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ftest changes LGTM

@github-actions
Copy link
Copy Markdown

Ticket title is 'daos_test/rebuild.py:DaosCoreTestRebuild.test_rebuild_31 and test_rebuild_32 failures'
Status is 'In Review'
Labels: 'ci_master_daily,daily_test'
https://daosio.atlassian.net/browse/DAOS-16575

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.

2 participants