Bug #13268
closedSecondary groups are not read /w SAMBA 4.2.2 & CEPHFS VFS module
0%
Description
Tested with a Windows 7 & 2008 client.
When I'm using the ceph_vfs module in SAMBA it seems secondary groups are not read for access to files and directories. It works when I use a kernel mount to cephfs which I share through SAMBA.
Also, ACLs (acl_xattr module) doesn't work either together with the ceph_vfs module.
Again, if I use a kernel mount for CEPHFS, ACLs work without any problems.
Updated by Zheng Yan over 8 years ago
secondary groups should work if you use the newest developing code. I'am working on posix ACL supporting.
Updated by Dennis Kramer over 8 years ago
You mean the development version of SAMBA? Which version?
Updated by Dennis Kramer over 8 years ago
I have cloned https://github.com/ceph/samba.git but it doesn't work with secondary groups
Updated by Dennis Kramer over 8 years ago
Any news when the code hits the stable tree?
Updated by Dennis Kramer over 8 years ago
Okay, very nice! What was the issue? I would've thought the issue would be in the samba ceph vfs module...
Updated by Nathan Cutler over 8 years ago
If this is a bugfix and you want it backported to hammer, please:
- update this ticket with the commit hash(es) and/or PR number(s) of the fix
- update the Backport field of this ticket (e.g. "hammer")
- if the PR has been merged already, the status of this ticket should be changed to "Pending Backport"
This is only for Ceph backports, of course. If the fix is in Samba, ignore this message :-)
Updated by Zheng Yan over 8 years ago
- Status changed from New to Pending Backport
- Target version set to v0.94.6
Updated by Nathan Cutler over 8 years ago
https://github.com/ceph/ceph/pull/6604 is a backport, to hammer, of https://github.com/ceph/ceph/commit/f04c8da5432174874ca97d11a5b2fef56f95d73d which is one of three commits in https://github.com/ceph/ceph/pull/5528
What about the other two commits in that PR? Should they be backported as well?
Updated by Nathan Cutler over 8 years ago
- Target version deleted (
v0.94.6)
Also, that PR claims to be fixing http://tracker.ceph.com/issues/12617 - what is the relationship between that issue and this one?
Updated by Loïc Dachary over 8 years ago
- Copied to Backport #13809: hammer: Secondary groups are not read /w SAMBA 4.2.2 & CEPHFS VFS module added
Updated by Zheng Yan over 8 years ago
Nathan Cutler wrote:
https://github.com/ceph/ceph/pull/6604 is a backport, to hammer, of https://github.com/ceph/ceph/commit/f04c8da5432174874ca97d11a5b2fef56f95d73d which is one of three commits in https://github.com/ceph/ceph/pull/5528
What about the other two commits in that PR? Should they be backported as well?
I squash them into one commit
Updated by Zheng Yan over 8 years ago
Nathan Cutler wrote:
Also, that PR claims to be fixing http://tracker.ceph.com/issues/12617 - what is the relationship between that issue and this one?
that one is for ceph-fuse. but the root cause is the same.
Updated by Dennis Kramer over 8 years ago
Does the ceph samba_vfs also needs to be fixed? Upgrading to Infernalis doesn't resolve my issue. ACL's (acl_xattr in samba) still not working.
Updated by Zheng Yan over 8 years ago
Dennis Kramer wrote:
Does the ceph samba_vfs also needs to be fixed? Upgrading to Infernalis doesn't resolve my issue. ACL's (acl_xattr in samba) still not working.
For ACL support, both libcephfs and samba require modification.
https://github.com/ceph/samba/tree/wip-acl
https://github.com/ceph/ceph/tree/wip-libcephfs-acl
Updated by Dennis Kramer over 8 years ago
Does these modifications only need to be applied on a ceph client?
Updated by Loïc Dachary about 8 years ago
- Status changed from Pending Backport to Resolved
Updated by Patrick Donnelly about 5 years ago
- Category deleted (
43) - Labels (FS) Samba/CIFS added