pacific: MClientRequest: properly handle ceph_mds_request_head_legacy for ext_num_retry, ext_num_fwd, owner_uid, owner_gid #54410
Conversation
…quest_head_legacy When a client is too old and uses struct ceph_mds_request_head_legacy we must fill new ext_num_retry and ext_num_fwd fields from an old num_retry and num_fwd. Fixes: ceph#45669 Fixes: https://tracker.ceph.com/issues/63288 Fixes: commit cbd7e30 ("ceph_fs.h: add 32 bits extended num_retry and num_fwd support") Signed-off-by: Alexander Mikhalitsyn <aleksandr.mikhalitsyn@canonical.com> (cherry picked from commit 43f32a4)
|
cc @vshankar |
|
@mihalicyn This PR is a partial backport of PR #54149. 2nd commit (a70a70f) from that PR is missing. I couldn't find an explanation for this in the commit message or on the tracker ticket. Can you please backport it too or, for future reference, add a note explaining why it wasn't backported? cc @vshankar |
Sure, have added to the backport tracker. And I already have this in the first message in this PR:
|
There was a problem hiding this comment.
CephFS approved - https://tracker.ceph.com/projects/cephfs/wiki/Pacific#22-Nov-2023
In this packport ("MClientRequest: handle owner_uid and owner_gid from ceph_mds_request_head_legacy") commit was dropped because it is only relevant to the changes from #53137 pull request. But we decided not to take this change into the final pacific release
backport tracker: https://tracker.ceph.com/issues/63478
backport of #54149
parent tracker: https://tracker.ceph.com/issues/63288