[Bugs] [Bug 1312878] Glusterd: Creation of volume is failing if one of the brick is down on the server

bugzilla at redhat.com bugzilla at redhat.com
Thu Mar 3 04:16:23 UTC 2016


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

Atin Mukherjee <amukherj at redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|POST                        |MODIFIED



--- Comment #2 from Vijay Bellur <vbellur at redhat.com> ---
COMMIT: http://review.gluster.org/13550 committed in release-3.7 by Atin
Mukherjee (amukherj at redhat.com) 
------
commit cdc96d5d4c6247d9b0ca942eeb37338dacfe93ee
Author: Atin Mukherjee <amukherj at redhat.com>
Date:   Tue Jan 19 10:45:22 2016 +0530

    glusterd: use string comparison for realpath checks in
glusterd_is_brickpath_available

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

    glusterd_is_brickpath_available () used to call realpath() for checking the
    whether the new brick path matches with the existing ones. The problem with
this
    is if the underlying file system is bad for any one of the existing bricks
then
    realpath() would fail and we wouldn't allow to create the new brick even if
it
    should be allowed.

    Fix is to use string comparison with having a new field real_path in
brickinfo
    to store the absolute path

    Change-Id: I1250ea5345f00fca0f6128056ebd08750d604f0a
    BUG: 1312878
    Signed-off-by: Atin Mukherjee <amukherj at redhat.com>
    Reviewed-on: http://review.gluster.org/13258
    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: Jeff Darcy <jdarcy at redhat.com>
    Reviewed-on: http://review.gluster.org/13550

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


More information about the Bugs mailing list