squid: client: Fix leading / issue with mds_check_access#58981
squid: client: Fix leading / issue with mds_check_access#58981kotreshhr wants to merge 3 commits intoceph:squidfrom
Conversation
|
jenkins test make check |
|
jenkins test api |
|
jenkins test make check |
vshankar
left a comment
There was a problem hiding this comment.
@kotreshhr PTAL at the failure.
Signed-off-by: Rishabh Dave <ridave@redhat.com> (cherry picked from commit 7626bd5)
Fixes: https://tracker.ceph.com/issues/67212 Signed-off-by: Kotresh HR <khiremat@redhat.com> (cherry picked from commit 983f893)
The "Client::mds_check_access" expects the target_path without leading '/' as it eventually calls the "MDSCapMatch::match_path" which expects the target_path passed to be with out leading '/' as well. The single leading '/' was being removed. But absolute path constructed did have leading '//', so removing all the leading '/' was necessary. This causes the clients not to be able to access a particular path even though it has a rw permission on the specific path. The patche fixes the leading '//' issue. Fixes: https://tracker.ceph.com/issues/67212 Signed-off-by: Kotresh HR <khiremat@redhat.com> (cherry picked from commit 2e2adb2)
1971856 to
55373f9
Compare
|
jenkins test api |
|
This PR is under test in https://tracker.ceph.com/issues/68851. |
|
This pull request has been automatically marked as stale because it has not had any activity for 60 days. It will be closed if no further activity occurs for another 30 days. |
|
This pull request has been automatically closed because there has been no activity for 90 days. Please feel free to reopen this pull request (or open a new one) if the proposed change is still appropriate. Thank you for your contribution! |
|
jenkins test make check |
|
The PR #65359 takes care of this backport |
backport tracker: https://tracker.ceph.com/issues/67293
backport of #58878
parent tracker: https://tracker.ceph.com/issues/67212
Also backported a required qa fix 7626bd5
this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh