Mohit, Can you set up a call/meeting where you can explain the current patch? That would make it easier to review.
Thanks, Nithya On 13 July 2017 at 10:32, Raghavendra Gowdappa <[email protected]> wrote: > All, > > Patch [1] is getting more complex day by day. We had to extend permission > healing to xattr healing as 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 [email protected] http://lists.gluster.org/mailman/listinfo/gluster-devel
