[Bugs] [Bug 1420993] Modified volume options not synced once offline nodes comes up.

[bugzilla at redhat.com]

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



--- Comment #2 from Worker Ant <bugzilla-bot at gluster.org> ---
COMMIT: https://review.gluster.org/16594 committed in release-3.8 by Atin
Mukherjee (amukherj at redhat.com) 
------
commit dae553d2538cf465edfff567909b846025762a3e
Author: Atin Mukherjee <amukherj at redhat.com>
Date:   Thu Feb 9 12:56:38 2017 +0530

    glusterd: ignore return code of glusterd_restart_bricks

    When GlusterD is restarted on a multi node cluster, while syncing the
    global options from other GlusterD, it checks for quorum and based on
    which it decides whether to stop/start a brick. However we handle the
    return code of this function in which case if we don't want to start any
    bricks the ret will be non zero and we will end up failing the import
    which is incorrect.

    Fix is just to ignore the ret code of glusterd_restart_bricks ()

    >Reviewed-on: https://review.gluster.org/16574
    >Smoke: Gluster Build System <jenkins at build.gluster.org>
    >NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    >CentOS-regression: Gluster Build System <jenkins at build.gluster.org>
    >Reviewed-by: Samikshan Bairagya <samikshan at gmail.com>
    >Reviewed-by: Jeff Darcy <jdarcy at redhat.com>
    >(cherry picked from commit 55625293093d485623f3f3d98687cd1e2c594460)

    Change-Id: I37766b0bba138d2e61d3c6034bd00e93ba43e553
    BUG: 1420993
    Signed-off-by: Atin Mukherjee <amukherj at redhat.com>
    Reviewed-on: https://review.gluster.org/16594
    Smoke: Gluster Build System <jenkins at build.gluster.org>
    Reviewed-by: Samikshan Bairagya <samikshan at gmail.com>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.org>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>

-- 
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=5kbglk8Kwd&a=cc_unsubscribe