Skip to content

DAOS-17843 rebuild: add some logs for rebuild enumerate #17309

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.

Sign up for GitHub

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

Jump to bottom
Merged
gnailzenh merged 1 commit into from
Dec 31, 2025
Merged

DAOS-17843 rebuild: add some logs for rebuild enumerate #17309

gnailzenh merged 1 commit into from
Dec 31, 2025

Conversation

@liuxuezhao
Copy link
Contributor

@liuxuezhao liuxuezhao commented Dec 24, 2025 •
edited
Loading

  1. add some DEBUG logs for rebuild enumerate
  2. add WARN log if some engines did not report EC agg epoch progress in 600S.

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

@liuxuezhao
DAOS-17843 rebuild: add some logs for rebuild enumerate
8382e6c 
1. add some DEBUG logs for rebuild enumerate
2. add WARN log if some engines did not report EC agg epoch progress in
   600S.
3. Fix a typo for sub_anchors->sa_nr compare

Signed-off-by: Xuezhao Liu <[email protected]>
@liuxuezhao liuxuezhao requested review from a team as code owners December 24, 2025 04:24
@github-actions
Copy link

github-actions bot commented Dec 24, 2025

Ticket title is 'Pool rebuild stuck in pulling for 5+ hours'
Status is 'In Progress'
Labels: 'ALCF,hpe_cluster'
https://daosio.atlassian.net/browse/DAOS-17843

@liuxuezhao
Copy link
Contributor Author

liuxuezhao commented Dec 29, 2025

this PR merged to #17324, if it could not land before PR17324, will be landed together with it.

@gnailzenh gnailzenh merged commit d9ae7df into master Dec 31, 2025
43 checks passed
@gnailzenh gnailzenh deleted the lxz/rb_enum_dbg_master branch December 31, 2025 05:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@NiuYawei NiuYawei NiuYawei approved these changes

@gnailzenh gnailzenh gnailzenh approved these changes

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@liuxuezhao @NiuYawei @gnailzenh