Skip to content

DAOS-17322 ddb: DTX aggregation with ddb (part 3) - #17022

Merged
daltonbohning merged 7 commits into
masterfrom
ckochhof/dev/master/daos-17322-part3
Dec 3, 2025
Merged

daltonbohning merged 7 commits into
masterfrom
ckochhof/dev/master/daos-17322-part3

Conversation

@knard38

@knard38 knard38 commented Oct 23, 2025 •

Copy link
Copy Markdown
Contributor

Description

Fix reviewers comments of the PR #16746:

  • Fix spelling issue
  • Fix code style

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

Fix spelling issue and code style.

Features: aggregation
Signed-off-by: Cedric Koch-Hofer <cedric.koch-hofer@hpe.com>
@knard38 knard38 self-assigned this Oct 23, 2025
@knard38 knard38 added the CR Catastrophic Recovery Feature label Oct 23, 2025
@github-actions

Copy link
Copy Markdown

Ticket title is 'DTX aggregation with ddb'
Status is 'Resolved'
Labels: '2.8pp'
https://daosio.atlassian.net/browse/DAOS-17322

@knard38 knard38 changed the title DAOS-17322 ddb: DTX aggregation with ddb DAOS-17322 ddb: DTX aggregation with ddb (part 3) Oct 23, 2025
@knard38
knard38 marked this pull request as ready for review October 24, 2025 05:25
@knard38
knard38 requested review from a team as code owners October 24, 2025 05:25
@daosbuild3

Copy link
Copy Markdown
Collaborator

Test stage Functional Hardware Large MD on SSD completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net//job/daos-stack/job/daos/view/change-requests/job/PR-17022/1/execution/node/1421/log

@daosbuild3

Copy link
Copy Markdown
Collaborator

Test stage Functional Hardware Medium Verbs Provider MD on SSD completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net//job/daos-stack/job/daos/view/change-requests/job/PR-17022/1/execution/node/1376/log

@daosbuild3

Copy link
Copy Markdown
Collaborator

Test stage Build on Leap 15.5 with Intel-C and TARGET_PREFIX completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net//job/daos-stack/job/daos/view/change-requests/job/PR-17022/2/execution/node/296/log

@daosbuild3

Copy link
Copy Markdown
Collaborator

@daosbuild3

Copy link
Copy Markdown
Collaborator

@daosbuild3

Copy link
Copy Markdown
Collaborator

@daosbuild3

Copy link
Copy Markdown
Collaborator

Test stage Functional Hardware Large MD on SSD completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net//job/daos-stack/job/daos/view/change-requests/job/PR-17022/3/execution/node/1352/log

@daosbuild3

Copy link
Copy Markdown
Collaborator

Test stage Functional Hardware Medium MD on SSD completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net//job/daos-stack/job/daos/view/change-requests/job/PR-17022/3/execution/node/1380/log

@daosbuild3

Copy link
Copy Markdown
Collaborator

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

@knard38

knard38 commented Oct 29, 2025

Copy link
Copy Markdown
Contributor Author

@kjacque and @gnailzenh , please could you have a look at this PR.
Thanks in advance for your attention.

@daosbuild3

Copy link
Copy Markdown
Collaborator

Test stage Functional Hardware Medium Verbs Provider MD on SSD completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net//job/daos-stack/job/daos/view/change-requests/job/PR-17022/4/execution/node/1352/log

@knard38

knard38 commented Oct 30, 2025 •

Copy link
Copy Markdown
Contributor Author

Only the following functional test daos_test/suite.py:test_daos_ec_io failed during the last run of the CI.

After some investigation, this failure is not related to this PR but to the Pool Map management.
Indeed an assertion failure was raised when trying to lock a mutex in the function dc_tx_check_pmv_internal() at line 685
image
This is confirmed with the following log of the engine of rank 5:

2025/10/29 22:58:34.197428 hdr-124 DAOS[126292/7/9] object EMRG src/object/obj_tx.c:685 dc_tx_check_pmv_internal() Assertion '_rc == 0' failed: pthread_mutex_timedlock(0x7f1c48a1acd8) rc=22 Invalid argument

More details could be found in the ticket https://daosio.atlassian.net/browse/DAOS-18168

@knard38
knard38 requested a review from a team October 30, 2025 10:12
@knard38 knard38 added the forced-landing The PR has known failures or has intentionally reduced testing, but should still be landed. label Oct 30, 2025
@knard38

knard38 commented Oct 30, 2025

Copy link
Copy Markdown
Contributor Author

@daos-stack/daos-gatekeeper , please could you land this PR as the only error occurring during the CI validation is not related to this PR. More details in my previous comment #17022 (comment)

@daosbuild3

Copy link
Copy Markdown
Collaborator

@daosbuild3

Copy link
Copy Markdown
Collaborator

@daosbuild3

Copy link
Copy Markdown
Collaborator

Test stage Functional Hardware Medium MD on SSD completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net//job/daos-stack/job/daos/view/change-requests/job/PR-17022/8/execution/node/1321/log

@daosbuild3

Copy link
Copy Markdown
Collaborator

@knard38

knard38 commented Dec 3, 2025

Copy link
Copy Markdown
Contributor Author

@daos-stack/daos-gatekeeper please could you lend this PR or tell me what is missing.

@knard38 knard38 removed the forced-landing The PR has known failures or has intentionally reduced testing, but should still be landed. label Dec 3, 2025
@daltonbohning
daltonbohning merged commit 7ac9fa0 into master Dec 3, 2025
42 checks passed
@daltonbohning
daltonbohning deleted the ckochhof/dev/master/daos-17322-part3 branch December 3, 2025 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CR Catastrophic Recovery Feature

Development

Successfully merging this pull request may close these issues.

6 participants