Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

quincy: mds: inode_t flags may not be protected by the policylock during set_vxattr #57178

Open
wants to merge 1 commit into
base: quincy
Choose a base branch
from

Conversation

batrick
Copy link
Member

@batrick batrick commented Apr 30, 2024

Note: this backport omits testing commits because qa/tasks/cephfs/test_subvolume.py does not exist in quincy.

backport tracker: https://tracker.ceph.com/issues/65713


backport of #56934
parent tracker: https://tracker.ceph.com/issues/65496

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

The code logic had a serious defect that it would only execute xlock_policylock
once such that a retry would then proceed to executing the vxattr setting.

Fixes: https://tracker.ceph.com/issues/65496
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit 2931860)

Conflicts:
	src/mds/Server.cc: no quiesce in quincy
@batrick batrick added this to the quincy milestone Apr 30, 2024
@batrick batrick added the cephfs Ceph File System label Apr 30, 2024
@batrick
Copy link
Member Author

batrick commented May 1, 2024

jenkins test api

@batrick
Copy link
Member Author

batrick commented May 1, 2024

jenkins test windows

@batrick
Copy link
Member Author

batrick commented May 1, 2024

jenkins test make check

@batrick
Copy link
Member Author

batrick commented May 1, 2024

jenkins test readthedocs

@batrick
Copy link
Member Author

batrick commented May 1, 2024

jenkins render docs

@batrick
Copy link
Member Author

batrick commented May 21, 2024

jenkins test make check

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cephfs Ceph File System
Projects
None yet
1 participant