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

[bugzilla at redhat.com]

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



--- Comment #4 from Vijay Bellur <vbellur at redhat.com> ---
COMMIT: http://review.gluster.org/14302 committed in master by Pranith Kumar
Karampuri (pkarampu at redhat.com) 
------
commit 7dc5d73410f0e9f846c593887637001ca43bc4a0
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

    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: 1335429
    Change-Id: I1be7f9defef2ae03be7eec8d7d49bf34adeca82c
    Signed-off-by: Pranith Kumar K <pkarampu at redhat.com>
    Reviewed-on: http://review.gluster.org/14302
    Reviewed-by: Krutika Dhananjay <kdhananj at redhat.com>
    Reviewed-by: Anuradha Talur <atalur at redhat.com>
    Smoke: Gluster Build System <jenkins at build.gluster.com>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    Reviewed-by: Ravishankar N <ravishankar at redhat.com>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.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=7zN3sqdmHw&a=cc_unsubscribe