Skip to content

DAOS-18705 rebuild: set rebuild flag before creating rebuild_pool_tls (#17941)#17962

Merged
gnailzenh merged 13 commits intorelease/2.6from
liang/b2_6_agg_epc
Apr 17, 2026
Merged

DAOS-18705 rebuild: set rebuild flag before creating rebuild_pool_tls (#17941)#17962
gnailzenh merged 13 commits intorelease/2.6from
liang/b2_6_agg_epc

Conversation

@gnailzenh
Copy link
Copy Markdown
Collaborator

@gnailzenh gnailzenh commented Apr 9, 2026

  • set rebuild flag before creating rebuild_pool_tls, otherwise aggregation
    can progress to higher epoch than rebuild.
  • aggregation doesn't do full scan anymore after rebuild
  • fix a rpt refcount leak in rebuild_tgt_scan_handler()

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

- stop refreshing aggregation epoch while rebuilding
- set rebuilding flag before setting rebuild fence

Signed-off-by: Liang Zhen <gnailzenh@gmail.com>
Signed-off-by: Liang Zhen <gnailzenh@gmail.com>
@gnailzenh gnailzenh requested review from a team as code owners April 9, 2026 16:40
@gnailzenh gnailzenh changed the title Liang/b2 6 agg epc DAOS-18705 rebuild: set rebuild flag before creating rebuild_pool_tls (#17941) Apr 9, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2026

Ticket title is 'DAOS 2.6.5: Interrupt rebuild with reintegration and interrupt with exclude with active IO'
Status is 'In Progress'
Labels: 'test_2.6.5,testp1'
https://daosio.atlassian.net/browse/DAOS-18705

gnailzenh and others added 8 commits April 10, 2026 23:04
Signed-off-by: Liang Zhen <gnailzenh@gmail.com>
Signed-off-by: Liang Zhen <gnailzenh@gmail.com>
Signed-off-by: Liang Zhen <gnailzenh@gmail.com>
Signed-off-by: Liang Zhen <gnailzenh@gmail.com>
Signed-off-by: Liang Zhen <gnailzenh@gmail.com>
Signed-off-by: Liang Zhen <gnailzenh@gmail.com>
@gnailzenh gnailzenh added priority Ticket has high priority (automatically managed) approved-to-merge PR has received release branch merge approval labels Apr 14, 2026
@github-actions github-actions bot removed the priority Ticket has high priority (automatically managed) label Apr 14, 2026
@daosbuild3
Copy link
Copy Markdown
Collaborator

Signed-off-by: Liang Zhen <gnailzenh@gmail.com>
@daosbuild3
Copy link
Copy Markdown
Collaborator

@gnailzenh gnailzenh added the priority Ticket has high priority (automatically managed) label Apr 15, 2026
@daosbuild3
Copy link
Copy Markdown
Collaborator

Test stage Functional Hardware Medium Verbs Provider completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net//job/daos-stack/job/daos/view/change-requests/job/PR-17962/10/execution/node/1476/log

@daosbuild3
Copy link
Copy Markdown
Collaborator

Signed-off-by: Wang Shilong <shilong.wang@hpe.com>
@github-actions github-actions bot removed the priority Ticket has high priority (automatically managed) label Apr 16, 2026
@gnailzenh gnailzenh requested a review from NiuYawei April 17, 2026 01:55
@gnailzenh gnailzenh merged commit 62698a1 into release/2.6 Apr 17, 2026
49 of 50 checks passed
@gnailzenh gnailzenh deleted the liang/b2_6_agg_epc branch April 17, 2026 02:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved-to-merge PR has received release branch merge approval

Development

Successfully merging this pull request may close these issues.

6 participants