Bug 1091936

Summary: Incase of ACL not set on a file, nfs4_getfacl should return a default acl
Product: [Red Hat Storage] Red Hat Gluster Storage Reporter: Soumya Koduri <skoduri>
Component: nfs-ganeshaAssignee: Jiffin <jthottan>
Status: CLOSED ERRATA QA Contact: Saurabh <saujain>
Severity: low Docs Contact:
Priority: medium    
Version: 2.1CC: ansubram, asriram, mzywusko, ndevos, nlevinki, saujain, skoduri, vagarwal
Target Milestone: ---Keywords: ZStream
Target Release: RHGS 3.1.1   
Hardware: All   
OS: All   
Whiteboard:
Fixed In Version: glusterfs-3.7.1-12 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-10-05 07:07:40 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:    
Bug Blocks: 1087818, 1251815    

Description Soumya Koduri 2014-04-28 10:57:47 UTC
Description of problem:
Recently NFSv4 ACL support was added to NFS-ganesha for FSAL_GLUSTER. But while using that support, incase if there is no ACL set, and the client issues nfs4_getfacl to read the ACL, we return empty response to the client.

Instead we should be constructing a default_acl from the mode-bits and send it to the client as done by Kernel-NFS. 


Version-Release number of selected component (if applicable):
ACLs supported for NFS-Ganesha - FSAL_GLUSTER from version 2.1 onwards.

Comment 2 Soumya Koduri 2014-07-07 11:01:21 UTC
This is not a major issue but a tracker bug opened by dev. This only helps us better the ACL usage experience.  
This will be addressed for the next release.

Comment 3 Shalaka 2014-09-20 16:37:54 UTC
Please review and sign-off edited doc text.

Comment 4 Shalaka 2014-09-24 07:07:18 UTC
Soumya reviewed the text during online review meeting. Hence removing need_info.

Comment 10 Saurabh 2015-08-25 11:04:46 UTC
As can be seen that the default is available,
[root@rhsauto004 ~]# nfs4_getfacl /mnt/run6067/
A::OWNER@:rwaDxtTcCy
A::GROUP@:rxtcy
A::EVERYONE@:rxtcy

Hence moving this bz to verified.

Comment 13 errata-xmlrpc 2015-10-05 07:07:40 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHSA-2015-1845.html