[Bugs] [Bug 1344561] conservative merge happening on a x3 volume for a deleted file

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1344561



--- Comment #2 from Vijay Bellur <vbellur at redhat.com> ---
COMMIT: http://review.gluster.org/14688 committed in release-3.7 by Pranith
Kumar Karampuri (pkarampu at redhat.com) 
------
commit 840bc1209c942f110037065ece4a52dce04d36c5
Author: Ravishankar N <ravishankar at redhat.com>
Date:   Wed Jun 1 17:57:41 2016 +0530

    afr: Consider ENOSPC and EDQUOT as symmetric errors

    Backport of http://review.gluster.org/#/c/14604/

    Problem:
    Since commit 8eaa3506ead4f11b81b146a9e56575c79f3aad7b, in replica 3, if a
    brick is down and a create fails on the other 2 brick with EDQUOT, we
consider
    it an unsymmetric error and hence do not do post-op. So the dirty xattr
    remains set on the parent dir, leading to conservative merges during heal
when
    all bricks are up. i.e. a file deleted on the source might re-appear after
heal.

    Fix:
    Consider ENOSPC and  EDQUOT as symmetric errors since there is no
    possibility of partial inode or entry modification operations possible when
    quota is enabled. IOW, if quota reports EDQUOT, the no. of bytes written
    (or not written) will be the same on all bricks of the replica.
    Likewise, the entry operation (create, mkdir...) will either succeed or
    not succeed on all bricks.

    Change-Id: Iacb1108e9ef4a918e36242fb4a957455133744e9
    BUG: 1344561
    Signed-off-by: Ravishankar N <ravishankar at redhat.com>
    Reviewed-on: http://review.gluster.org/14688
    Smoke: Gluster Build System <jenkins at build.gluster.com>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.com>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    Reviewed-by: Pranith Kumar Karampuri <pkarampu at redhat.com>

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=4fsXcalpr9&a=cc_unsubscribe