----- Original Message ----- > From: "Raghavendra Gowdappa" <rgowdapp@xxxxxxxxxx> > To: "Gluster Devel" <gluster-devel@xxxxxxxxxxx> > Cc: "Gluster Devel" <gluster-devel@xxxxxxxxxxx>, "Mohit Agrawal" <moagrawa@xxxxxxxxxx> > Sent: Thursday, July 13, 2017 10:32:28 AM > Subject: 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@xxxxxxxxxxx > http://lists.gluster.org/mailman/listinfo/gluster-devel > _______________________________________________ Gluster-devel mailing list Gluster-devel@xxxxxxxxxxx http://lists.gluster.org/mailman/listinfo/gluster-devel
