[Gluster-devel] On change #15468

[Raghavendra Gowdappa]

----- Original Message -----
> From: "Raghavendra Gowdappa" <rgowdapp at redhat.com>
> To: "Gluster Devel" <gluster-devel at gluster.org>
> Cc: "Gluster Devel" <gluster-devel at gluster.org>, "Mohit Agrawal" <moagrawa at redhat.com>
> Sent: Thursday, July 13, 2017 10:32:28 AM
> Subject: [Gluster-devel] On change #15468
> 
> All,
> 
> Patch [1] is getting more complex day by day. We had to extend permission
> healing to xattr healing as 

s/permission healing to xattr healing/permission healing to the same logic as xattr healing/

> Mohith saw some regressions in uid/gid healing
> codepath. So, I would request you to take a look and comment.
> 
> Current behavior of the patch:
> * MDS (Meta data server) is the source of truth (among other subvols
> containing directory) for user, posix-acls, some other xattrs, uid/gid on a
> directory
> * If mds is not there, choose existing hashed-subvol as mds. Once chosen mds
> remains constant till that brick is removed.
> * non-availability of mds leads to failure of setxattr/setattr operation.
> Some volume level operations like add-brick/remove-brick maintain metadata
> on root, it is necessary that these operations don't fail. So, no mds is
> chosen for root and existing behavior (doing ops on all subvols
> simultaneously) is retained. Also, note that root won't have any
> hashed-subvol (we could've chose first child of dht as mds, but didn't for
> reasons mentioned earlier)
> * uid/gid healing logic is changed from having subvol with latest ctime as
> source of truth to mds being source of truth.
> 
> [1] https://review.gluster.org/15468
> 
> regards,
> Raghavendra
> _______________________________________________
> Gluster-devel mailing list
> Gluster-devel at gluster.org
> http://lists.gluster.org/mailman/listinfo/gluster-devel
>