[Gluster-devel] [NFS-Ganesha-Devel] Re: Problems about cache virtual glusterfs ACLs for ganesha in md-cache

[Soumya Koduri]

On 10/12/18 12:04 PM, Soumya Koduri wrote:
>>> 1. Cache it separately as posix ACLs (a new option maybe 
>>> "cache-glusterfs-acl" is added);
>>>      And make sure _posix_xattr_get_set fills them when lookup requests.
>>>
>>
>> I am not sure if posix layer can handle it. Virtual xattrs are 
>> in-memory and not stored on disk. They are converted to/from posix-acl 
>> in posix-acl xlator. So FWIU, posix-acl xlator should handle setting 
>> these attributes as part of LOOKUP response if needed. Same shall 
>> apply for any virtual xattr cached in md-cache. Request Poornima to 
>> comment.
> 
> Posix-acl can hand it correctly now.

Okay.

> 
>>
>> At a time, any gfapi consumer would use either posix-acl or virtual 
>> glusterfs ACLs. So having two options to selectively choose which one 
>> of them to cache sounds better to me instead of unnecessarily storing 
>> two different representations of the same ACL.
> 
> Make sense.
> I will add another option for virtual glusterfs ACLs in md-cache.

Cool. thanks!

-Soumya

> 
> thanks,
> Kinglong Mee