Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DAOS-17142 rebuild: exit rebuild_tgt_status_check_ult when RPT stale #15994

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

liuxuezhao
Copy link
Contributor

rebuild_tgt_status_check_ult() ULT only can exit when rebuild abort or globally done. When rebuild done, the leader notifies each target server by IV LAZY SYNC, the notification lost due to some network issue and lead to dangling rebuild_tgt_status_check_ult.

Steps for the author:

  • Commit message follows the guidelines.
  • Appropriate Features or Test-tag pragmas were used.
  • 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).

@liuxuezhao liuxuezhao requested review from a team as code owners February 28, 2025 10:27
Copy link

Ticket title is 'clean up rebuild_tgt_status_check() ULTs for globally-done rebuilds whose leader to target engine IV updates failed'
Status is 'In Progress'
https://daosio.atlassian.net/browse/DAOS-17142

@liuxuezhao liuxuezhao force-pushed the lxz/tgt_check_ult branch 2 times, most recently from be51151 to 7a78480 Compare February 28, 2025 13:26
rebuild_tgt_status_check_ult() ULT only can exit when rebuild abort
or globally done. When rebuild done, the leader notifies each target
server by IV LAZY SYNC, the notification lost due to some network
issue and lead to dangling rebuild_tgt_status_check_ult.

Signed-off-by: Xuezhao Liu <[email protected]>
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.

1 participant