Skip to content

DAOS-19299 mercury: add patch to fix handling of canceled events - #18691

Open
soumagne wants to merge 1 commit into
masterfrom
soumagne/mercury_cancel
Open

DAOS-19299 mercury: add patch to fix handling of canceled events#18691
soumagne wants to merge 1 commit into
masterfrom
soumagne/mercury_cancel

Conversation

@soumagne

@soumagne soumagne commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

This would have caused RPCs to remain in flight until timeout was reached

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

@soumagne
soumagne requested a review from a team as a code owner July 21, 2026 16:38
@github-actions

github-actions Bot commented Jul 21, 2026 •
edited
Loading

Copy link
Copy Markdown

Ticket title is 'DaosCoreTest.test_daos_extend_simple hangs (SIGTERM/timeout) in dfs_extend_fetch_kill()'
Status is 'In Review'
Labels: 'request_for_2.8.2'
https://daosio.atlassian.net/browse/DAOS-19299

@daosbuild3

daosbuild3 commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

Test stage Unit Test with memcheck completed with status UNSTABLE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos//view/change-requests/job/PR-18691/1/testReport/

@knard38 knard38 mentioned this pull request Jul 22, 2026
Draft
6 tasks
frostedcmos
frostedcmos previously approved these changes Jul 24, 2026
View reviewed changes
@soumagne
soumagne force-pushed the soumagne/mercury_cancel branch from b483dee to 017ee0e Compare August 10, 2026 22:50
@daosbuild3

daosbuild3 commented Aug 10, 2026

Copy link
Copy Markdown
Collaborator

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

@soumagne
soumagne force-pushed the soumagne/mercury_cancel branch from 017ee0e to aaa157d Compare August 11, 2026 21:27
@daosbuild3

daosbuild3 commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

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

@daosbuild3

daosbuild3 commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

Test stage Functional on EL 9 completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net//job/daos-stack/job/daos/view/change-requests/job/PR-18691/3/execution/node/1268/log

@soumagne
soumagne requested a review from frostedcmos August 12, 2026 16:12
frostedcmos
frostedcmos previously approved these changes Aug 12, 2026
View reviewed changes
@soumagne
soumagne force-pushed the soumagne/mercury_cancel branch 2 times, most recently from 3151e76 to 660beff Compare August 18, 2026 15:39
@soumagne
soumagne requested a review from frostedcmos August 20, 2026 15:47
frostedcmos
frostedcmos previously approved these changes Aug 20, 2026
View reviewed changes
@soumagne
soumagne force-pushed the soumagne/mercury_cancel branch 2 times, most recently from a3785c4 to b7664fa Compare September 2, 2026 20:09
@soumagne
soumagne force-pushed the soumagne/mercury_cancel branch from b7664fa to 6a8a779 Compare October 27, 2026 22:47
@soumagne
DAOS-19299 mercury: add patch to fix handling of canceled events
38eedcd 
This would have caused RPCs to remain in flight until timeout was reached

Skip-stack-optimization: true

Signed-off-by: Jerome Soumagne <jerome.soumagne@hpe.com>
@soumagne
soumagne force-pushed the soumagne/mercury_cancel branch from 6a8a779 to 38eedcd Compare October 29, 2026 20:22

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@frostedcmos frostedcmos frostedcmos left review comments
@jgmoore-or jgmoore-or Awaiting requested review from jgmoore-or

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@soumagne @daosbuild3 @frostedcmos