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

mds: allow client to request caps when opening file #7952

Merged
merged 2 commits into from Mar 25, 2016

Conversation

ukernel
Copy link
Contributor

@ukernel ukernel commented Mar 7, 2016

Allow client to request xattrs while opening file. MDS adds xattrs
and corresponding caps to the reply. This is important for selinux
on kcephfs.

Fixes: #14360
Signed-off-by: Yan, Zheng zyan@redhat.com

Allow client to request xattrs while opening file. MDS adds xattrs
and corresponding caps to the reply. This is important for selinux
on kcephfs.

Fixes: ceph#14360
Signed-off-by: Yan, Zheng <zyan@redhat.com>
@ukernel ukernel added feature cephfs Ceph File System labels Mar 7, 2016
@gregsfortytwo gregsfortytwo self-assigned this Mar 7, 2016
@gregsfortytwo
Copy link
Member

This needs a qa-suite test.

Also, it doesn't seem to include the bits for actually sending back xattrs on an open?

@ukernel
Copy link
Contributor Author

ukernel commented Mar 10, 2016

the new line "mdr->getattr_caps = mask" in handle_client_open() does the trick

@gregsfortytwo
Copy link
Member

Oh, because that's used in encode_inodestat() to include the xattrs on reply.

Still needs a test though. :)

Signed-off-by: Yan, Zheng <zyan@redhat.com>
@ukernel
Copy link
Contributor Author

ukernel commented Mar 15, 2016

ceph/ceph-qa-suite#876

@gregsfortytwo
Copy link
Member

@gregsfortytwo
Copy link
Member

Oh, didn't read closely enough. Looks like it just needs to run against a different conf file.

2016-03-16T13:15:26.261 INFO:tasks.cephfs_test_runner:Only have 1 clients, require 2

@gregsfortytwo
Copy link
Member

Pushed a quick change moving the test into multiclient and will test again.

@gregsfortytwo gregsfortytwo merged commit 74ed53b into ceph:master Mar 25, 2016
@ukernel ukernel deleted the wip-14360 branch April 13, 2016 02:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants