Skip to content

DAOS-19381 rebuild: skip -1 target in layout_find_diff - #18754

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

DAOS-19381 rebuild: skip -1 target in layout_find_diff#18754
liuxuezhao wants to merge 1 commit into
release/2.8from
lxz/new_tgt_skip_28

Conversation

@liuxuezhao

Copy link
Copy Markdown
Contributor

When no spare target available, the new_tgt possibly be -1 in POST_REBUILD layout, skip that shard's remap to avoid such assertion - placement/jump_map.c:207 layout_find_diff() Assertion 'rc == 1' failed (pool_map_find_target(jmap->jmp_map.pl_poolmap, new_tgt, &new_pot) failed).

Add some debug log about rebuild target list.

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
liuxuezhao requested review from a team as code owners July 30, 2026 09:30
@github-actions

github-actions Bot commented Jul 30, 2026

Copy link
Copy Markdown

Ticket title is 'Aurora engine crash - determine_valid_spares() Assertion'
Status is 'In Progress'
Labels: 'test_2.8.0rc'
Job should run at elevated priority (1)
https://daosio.atlassian.net/browse/DAOS-19381

wangshilong
wangshilong previously approved these changes Jul 30, 2026
When no spare target available, the new_tgt possibly be -1 in
POST_REBUILD layout, skip that shard's remap to avoid such assertion -
placement/jump_map.c:207 layout_find_diff() Assertion 'rc == 1' failed
(pool_map_find_target(jmap->jmp_map.pl_poolmap, new_tgt, &new_pot)
failed).

Add some debug log about rebuild target list.

Signed-off-by: Xuezhao Liu <xuezhao.liu@hpe.com>
Co-authored-by: Wang Shilong <shilong.wang@hpe.com>
@liuxuezhao
liuxuezhao force-pushed the lxz/new_tgt_skip_28 branch from 54aa53e to aaad2ac Compare August 2, 2026 07:55
@github-actions github-actions Bot added the priority Ticket has high priority (automatically managed) label Aug 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority Ticket has high priority (automatically managed)

Development

Successfully merging this pull request may close these issues.

2 participants