Skip to content

DAOS-16952 cart: Ignore expired bulks and rpcs - #16898

Open
frostedcmos wants to merge 4 commits into
release/2.6from
frostedcmos/daos-16952-2.6-ignore-expired-bulks-and-rpcs
Open

frostedcmos wants to merge 4 commits into
release/2.6from
frostedcmos/daos-16952-2.6-ignore-expired-bulks-and-rpcs

Conversation

@frostedcmos

Copy link
Copy Markdown
Contributor

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

* DAOS-16251 cart: Ignore expired bulks and rpcs

- Switch rpc headers to transfer deadline instead of a timeout
- Add checks at the start and end of bulk transfer to ensure deadline has not expired.
- Add deadline expiration checks in all places where rpc_priv timeout is initialized

Signed-off-by: Alexander A Oganezov <alexander.a.oganezov@intel.com>
Signed-off-by: Alexander A Oganezov <alexander.oganezov@hpe.com>
Deadlines were not set for some internal RPCs due to conversion
from timeout to deadline happening in an rpc tracking code.
Some internal RPCs bypass rpc tracking

Signed-off-by: Alexander A Oganezov <alexander.oganezov@hpe.com>
@frostedcmos frostedcmos changed the title Frostedcmos/daos 16952 2.6 ignore expired bulks and rpcs DAOS-16952 cart: Ignore expired bulks and rpcs Sep 22, 2025
@frostedcmos frostedcmos added the unclean-cherry-pick Indicates that a cherry-pick had merge conflicts that needed resolving. label Sep 22, 2025
@github-actions

Copy link
Copy Markdown

Ticket title is 'CART: Change timeouts to deadlines'
Status is 'Awaiting backport'
Labels: 'request_for_2.6.5'
https://daosio.atlassian.net/browse/DAOS-16952

@daosbuild3

Copy link
Copy Markdown
Collaborator

@frostedcmos
frostedcmos marked this pull request as ready for review October 1, 2025 02:48
@frostedcmos
frostedcmos requested review from a team as code owners October 1, 2025 02:48
@frostedcmos
frostedcmos requested review from a team October 1, 2025 02:48
@frostedcmos
frostedcmos requested review from a team as code owners October 1, 2025 02:48

@daltonbohning daltonbohning left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ftest changes LGTM

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

Labels

unclean-cherry-pick Indicates that a cherry-pick had merge conflicts that needed resolving.

Development

Successfully merging this pull request may close these issues.

3 participants