Skip to content

DAOS-18766 container: use request RPC version for filter#18210

Open
wangshilong wants to merge 1 commit into
masterfrom
shilongw/DAOS-18766
Open

DAOS-18766 container: use request RPC version for filter#18210
wangshilong wants to merge 1 commit into
masterfrom
shilongw/DAOS-18766

Conversation

@wangshilong
Copy link
Copy Markdown
Contributor

  1. Pass the pool filter RPC version down to ds_cont_filter() and use it when reading container info so the returned layout matches the incoming request.
  2. remove stale ds_cont_rpc_protocol() lookups from the set-prop path, where request decoding already follows rpc->cr_opc.

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

1. Pass the pool filter RPC version down to ds_cont_filter() and use it when reading
container info so the returned layout matches the incoming request.
2. remove stale ds_cont_rpc_protocol() lookups from the set-prop path,
where request decoding already follows rpc->cr_opc.

Signed-off-by: Wang Shilong <shilong.wang@hpe.com>
@wangshilong wangshilong requested review from a team as code owners May 11, 2026 06:55
@github-actions
Copy link
Copy Markdown

Ticket title is 'ds_cont_set_prop_srv_handler should not take protocol version from ds_cont_rpc_protocol'
Status is 'In Review'
https://daosio.atlassian.net/browse/DAOS-18766

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.

1 participant