Bug 1226802

Summary: FSAL_GLUSTER : inherit ACLs is not working properly for group write permissions
Product: [Retired] nfs-ganesha Reporter: Jiffin <jthottan>
Component: FSAL_GLUSTERAssignee: Jiffin <jthottan>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: high Docs Contact:
Priority: high    
Version: develCC: jthottan, kkeithle, ndevos, skoduri, smohan, vagarwal
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: All   
Whiteboard:
Fixed In Version: nfs-ganesha-2.3-dev-7 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 1229569 (view as bug list) Environment:
Last Closed: 2016-02-17 06:57:45 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1215174    
Bug Blocks: 1229569    

Description Jiffin 2015-06-01 07:21:49 UTC 
Description of problem:

Files created in a directory that should inherit ACLs is not working for group write-permissions.That is specfic users in the cannot write into the file evenif he has nfs4_acl permission.

Version-Release number of selected component (if applicable):
mainline

How reproducible:
always

Steps to Reproduce:

1.Create and start a volume

2.Export a volume using nfs-ganesha

3.Create any user(user1) and group(group1) in the server and client

4.Mount the volume using nfs4 protocol

5.create directory at mount point : mkdir src

6.set inherit acl(default acl for directory) using nfs4_setfacl on src: nfs4_setfacl -a A:gdf:group1:RWX src 

7. Tries to write into a "file" inside src using user1 : su -c "/bin/echo 'Hello world' > src/file" user1

Actual results:
write fails for the file

Expected results:
write should succeed. 

Additional info:

In the case of kernel nfs it is working properly.The posix ACL set on the backend is same in both cases.But nfs4_acl for nfs-ganesha and kernel nfs is little bit different
Comment 1 Jiffin 2015-06-09 06:09:00 UTC 
In the nfs4_acl <-> posix_acl conversion , ACL entry EVERYONE should be considered for all other entry. This fix for this is send to ganesha uptream : https://review.gerrithub.io/#/c/235200/