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

smb_share.conf.j2: Add ignore system acls option to vfs_acl_xattr #82

Merged

Conversation

anoopcs9
Copy link
Collaborator

@anoopcs9 anoopcs9 commented Feb 5, 2024

For better coverage of smbtorture tests we require ignore system acls enabled for acl_xattr VFS module. This is done to successfully complete those tests where such a lossy translation attempt from NT ACLs to POSIX ACLs would result in failures by design.

fixes #81

@anoopcs9 anoopcs9 force-pushed the add-ignore-system-acls branch 2 times, most recently from c74ef1a to a5b4518 Compare February 5, 2024 08:20
Copy link
Collaborator

@xhernandez xhernandez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wouldn't it be better to make it configurable ?

We could also add support for configuring many other options.

@anoopcs9 anoopcs9 force-pushed the add-ignore-system-acls branch from a5b4518 to 6e41b20 Compare February 7, 2024 11:07
@anoopcs9 anoopcs9 marked this pull request as ready for review February 7, 2024 18:35
@anoopcs9 anoopcs9 requested a review from xhernandez February 8, 2024 10:51
@anoopcs9 anoopcs9 force-pushed the add-ignore-system-acls branch from 6e41b20 to 7d7d700 Compare February 8, 2024 13:23
Expanding the backend specific cluster settings we now introduce a
new key called "options" for already existing "samba_shares" where
one could define additonal smb.conf share options to be configured
with share definition.

Signed-off-by: Anoop C S <[email protected]>
For better coverage of smbtorture tests we require 'ignore system acls'
enabled for acl_xattr VFS module. This is done to successfully complete
those tests where such a lossy translation attempt from NT ACLs to POSIX
ACLs would result in failures by design.

Signed-off-by: Anoop C S <[email protected]>
@anoopcs9 anoopcs9 force-pushed the add-ignore-system-acls branch from 7d7d700 to 424d7c1 Compare February 9, 2024 08:49
@xhernandez xhernandez merged commit 57a9eac into samba-in-kubernetes:main Feb 9, 2024
6 checks passed
@anoopcs9 anoopcs9 deleted the add-ignore-system-acls branch March 11, 2024 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Possible inclusion of ignore system acls = yes option for vfs_acl_xattr
2 participants