Hi Neils,
Thanks for confirming this. I can take care of moving posix_acls and changing
glusterfs build scripts. I am not very sure about the CI though. It would be
great if someone can help me out on that.
Thanks,
Sriram
On 1/19/18, 4:17 PM, "Niels de Vos" <nde...@redhat.com> wrote:
On Mon, Jan 15, 2018 at 09:45:52AM -0800, Frank Filz wrote:
> Yes, we could move it into common code. I wish we had been able to
> avoid having it in the first place since it’s an imperfect conversion,
> but we have it so…
Moving it to common code seems like a good appraoch.
> If we move it into common code, I would appreciate someone throwing a
> fairly complete set of ACL tests against it and make sure all bugs are
> out, or at least that our conversions match what is done by knfsd.
Jiffin had quite a complete set of tests while he was working on the
conversion. We should probably add these to the CI.
> Is there any requirement for POSIX ACLs for NFS v3? Linux knfsd does
> have a sideband protocol for POSIX ACLs for NFS v3 which we COULD
> implement if there was demand for it.
I have seen requests for supporting ACLs in the Gluster/NFS (v3 only)
server. But those came at the time when NFSv4 was not much used yet. I
think there is no large demand for NFSv3 + ACLs, most users will be able
to use NFSv4 now.
Niels
>
>
>
> Frank
>
>
>
> From: Sriram Patil [mailto:srir...@vmware.com]
> Sent: Monday, January 15, 2018 3:32 AM
> To: nfs-ganesha-devel@lists.sourceforge.net
> Cc: Kamal Charan <kcha...@vmware.com>; Sakthi Kumar <sakt...@vmware.com>
> Subject: [Nfs-ganesha-devel] [GlusterFS] NFsv4 to POSIX ACL conversion
>
>
>
> Hi,
>
>
>
> I see that there is NFSv4 ACLs to POSIX ACLs mapping code written in
GlusterFS FSAL. Since, this is pretty standard code and other FSAls may use it
instead of duplicating the same code every time, can we move the ACL conversion
code to a common place in Ganesha? The code is independent of the other
GlusterFS code.
>
>
>
> Let me know your thoughts.
>
>
>
> Thanks,
>
> Sriram
>
>
>
> ---
> This email has been checked for viruses by Avast antivirus software.
>
https://urldefense.proofpoint.com/v2/url?u=https-3A__www.avast.com_antivirus&d=DwIDaQ&c=uilaK90D4TOVoH58JNXRgQ&r=3mYu-6ji6Qx1wg_jv5mtp8E_l5KQMHnNPIWlSioZ7IE&m=5gASqj2v9SyX55tOY13kONiwKcDB4JY9g411m02KclU&s=Tt7xnaw33gkp2koNqkpqjYqCtrLaBvhowodpIFVllk8&e=
>
------------------------------------------------------------------------------
> Check out the vibrant tech community on one of the world's most
> engaging tech sites, Slashdot.org!
https://urldefense.proofpoint.com/v2/url?u=http-3A__sdm.link_slashdot&d=DwIDaQ&c=uilaK90D4TOVoH58JNXRgQ&r=3mYu-6ji6Qx1wg_jv5mtp8E_l5KQMHnNPIWlSioZ7IE&m=5gASqj2v9SyX55tOY13kONiwKcDB4JY9g411m02KclU&s=z4SbVumW15qCN6dFOdRTOJJi7bHpR49yG-XQObhMtJk&e=
> _______________________________________________
> Nfs-ganesha-devel mailing list
> Nfs-ganesha-devel@lists.sourceforge.net
>
https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.sourceforge.net_lists_listinfo_nfs-2Dganesha-2Ddevel&d=DwIDaQ&c=uilaK90D4TOVoH58JNXRgQ&r=3mYu-6ji6Qx1wg_jv5mtp8E_l5KQMHnNPIWlSioZ7IE&m=5gASqj2v9SyX55tOY13kONiwKcDB4JY9g411m02KclU&s=Z8Fuswd882otYvsWlqJ2CP8ddTUHhqMdAo36F7D5uoY&e=
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel
