[Bugs] [Bug 1335433] Self heal shows different information for the same volume from each node

[bugzilla at redhat.com]

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



--- Comment #2 from Vijay Bellur <vbellur at redhat.com> ---
COMMIT: http://review.gluster.org/14303 committed in release-3.8 by Niels de
Vos (ndevos at redhat.com) 
------
commit 91dcbfb58ac06b84940d3de9049514465a3acd6b
Author: Pranith Kumar K <pkarampu at redhat.com>
Date:   Thu May 12 13:55:44 2016 +0530

    cluster/afr: Handle non-zero source in heal-info decision

            Backport of http://review.gluster.org/14302

    Problem:
    Spurious entries are reported in heal info when the mount is on
second/third
    brick of the replica pair because local-child is given preference in
selecting
    source. The code is supposed to suggest the file needs heal if the (source
< 0)
    (failure code path), but instead it is written as if any non-zero value
    is considered failure.

    Fix:
    Treat +ve source as success case

    BUG: 1335433
    Change-Id: Iede983b6560622964e91306405587da3f1de5748
    Signed-off-by: Pranith Kumar K <pkarampu at redhat.com>
    Reviewed-on: http://review.gluster.org/14303
    Reviewed-by: Krutika Dhananjay <kdhananj at redhat.com>
    Smoke: Gluster Build System <jenkins at build.gluster.com>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Ravishankar N <ravishankar at redhat.com>
    Reviewed-by: Anuradha Talur <atalur at redhat.com>
    Reviewed-by: Niels de Vos <ndevos at redhat.com>

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are on the CC list for the bug.