Skip to content

DAOS-17843 placement: correct the way JCH placement calls CRC - #16695

Merged
gnailzenh merged 9 commits into
release/2.6from
liang/b2_6_crc
Sep 25, 2025
Merged

gnailzenh merged 9 commits into
release/2.6from
liang/b2_6_crc

Conversation

@gnailzenh

Copy link
Copy Markdown
Collaborator

The Jump Consistent Hash (JCH) placement algorithm in DAOS calls the CRC functions incorrectly, which may lead to excessive hash collisions and imbalanced data distribution.
This patch corrects how the JCH placement algorithm invokes the CRC functions.

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

@gnailzenh
gnailzenh requested review from a team as code owners August 1, 2025 14:57
@github-actions

github-actions Bot commented Aug 1, 2025

Copy link
Copy Markdown

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

The Jump Consistent Hash (JCH) placement algorithm in DAOS calls the CRC
functions incorrectly, which may lead to excessive hash collisions and
imbalanced data distribution.
This patch corrects how the JCH placement algorithm invokes the CRC functions.

Signed-off-by: Zhen Liang <zhen.liang@hpe.com>
@daosbuild3

Copy link
Copy Markdown
Collaborator

@daosbuild3

Copy link
Copy Markdown
Collaborator

Test stage Unit Test bdev with memcheck on EL 8.8 completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos/job/PR-16695/2/display/redirect

@daosbuild3

Copy link
Copy Markdown
Collaborator

Test stage Unit Test with memcheck on EL 8.8 completed with status UNSTABLE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos//view/change-requests/job/PR-16695/2/testReport/

Signed-off-by: Wang Shilong <shilong.wang@hpe.com>
@daosbuild3

Copy link
Copy Markdown
Collaborator

Test stage Unit Test bdev with memcheck on EL 8.8 completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos/job/PR-16695/3/display/redirect

@daosbuild3

Copy link
Copy Markdown
Collaborator

@daosbuild3

Copy link
Copy Markdown
Collaborator

Test stage Functional Hardware Medium Verbs Provider completed with status UNSTABLE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos//view/change-requests/job/PR-16695/4/testReport/

@daosbuild3

Copy link
Copy Markdown
Collaborator

Test stage Functional Hardware Medium UCX Provider completed with status UNSTABLE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos//view/change-requests/job/PR-16695/4/testReport/

Upgrade test might create lower pool object version

Signed-off-by: Wang Shilong <shilong.wang@hpe.com>
@daosbuild3

Copy link
Copy Markdown
Collaborator

wangshilong
wangshilong previously approved these changes Sep 4, 2025
Signed-off-by: Wang Shilong <shilong.wang@hpe.com>
@daosbuild3

Copy link
Copy Markdown
Collaborator

Test stage Functional Hardware Medium Verbs Provider completed with status UNSTABLE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos//view/change-requests/job/PR-16695/8/testReport/

@daosbuild3

Copy link
Copy Markdown
Collaborator

@daosbuild3

Copy link
Copy Markdown
Collaborator

Signed-off-by: Wang Shilong <shilong.wang@hpe.com>
wangshilong
wangshilong previously approved these changes Sep 16, 2025
@daosbuild3

Copy link
Copy Markdown
Collaborator

@daosbuild3

Copy link
Copy Markdown
Collaborator

Test stage Functional Hardware Medium Verbs Provider completed with status UNSTABLE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos//view/change-requests/job/PR-16695/11/testReport/

Signed-off-by: Wang Shilong <shilong.wang@hpe.com>
@gnailzenh
gnailzenh merged commit ee13dff into release/2.6 Sep 25, 2025
43 of 45 checks passed
@gnailzenh
gnailzenh deleted the liang/b2_6_crc branch September 25, 2025 04:10
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.

3 participants