Bug #13268: Secondary groups are not read /w SAMBA 4.2.2 & CEPHFS VFS module - CephFS - Ceph

Actions

Copy link

Bug #13268

closed

Secondary groups are not read /w SAMBA 4.2.2 & CEPHFS VFS module

Added by Dennis Kramer over 8 years ago. Updated about 5 years ago.

Status:

Resolved

Priority:

Normal

Assignee:

Category:

Target version:

% Done:

Source:

other

Tags:

Backport:

hammer

Regression:

Severity:

3 - minor

Reviewed:

Affected Versions:

Ceph - v0.94.4

ceph-qa-suite:

samba

Component(FS):

Labels (FS):

Samba/CIFS

Pull request ID:

Crash signature (v1):

Crash signature (v2):

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.

Related issues 1 (0 open — 1 closed)

Actions

Copy link

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.

Actions

Copy link

Updated by Dennis Kramer over 8 years ago

You mean the development version of SAMBA? Which version?

Actions

Copy link

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

Actions

Copy link

Updated by Loïc Dachary over 8 years ago

Target version deleted (~~v0.94.4~~)

Actions

Copy link

Updated by Zheng Yan over 8 years ago

development version of ceph

Actions

Copy link

Updated by Dennis Kramer over 8 years ago

Any news when the code hits the stable tree?

Actions

Copy link

Updated by Zheng Yan over 8 years ago

the code is included in infernalis

Actions

Copy link

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...

Actions

Copy link

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 :-)

Actions

Copy link

#10

Updated by Zheng Yan over 8 years ago

Status changed from New to Pending Backport
Target version set to v0.94.6

https://github.com/ceph/ceph/pull/6604/files

Actions

Copy link

#11

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?

Actions

Copy link

#12

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?

Actions

Copy link

#13

Updated by Loïc Dachary over 8 years ago

Backport set to hammer

Actions

Copy link

#14

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

Actions

Copy link

#15

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

Actions

Copy link

#16