[Gluster-devel] On change #15468

[Nithya Balachandran]

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 <rgowdapp at redhat.com> 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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.gluster.org/pipermail/gluster-devel/attachments/20170713/37fec35c/attachment.html>