Bug 1266876

Patch posted for review :
http://review.gluster.org/#/c/12250/

This bug was accidentally moved from POST to MODIFIED via an error in automation, please see mmccune with any questions

REVIEW: http://review.gluster.org/12250 (glusterd : Introduce reset brick) posted (#5) for review on master by Anuradha Talur (atalur)

*** Bug 1316577 has been marked as a duplicate of this bug. ***

REVIEW: http://review.gluster.org/12250 (glusterd : Introduce reset brick) posted (#6) for review on master by Anuradha Talur (atalur)

REVIEW: http://review.gluster.org/12250 (glusterd : Introduce reset brick) posted (#7) for review on master by Anuradha Talur (atalur)

REVIEW: http://review.gluster.org/12250 (glusterd : Introduce reset brick) posted (#8) for review on master by Anuradha Talur (atalur)

REVIEW: http://review.gluster.org/12250 (glusterd : Introduce reset brick) posted (#9) for review on master by Anuradha Talur (atalur)

REVIEW: http://review.gluster.org/12250 (glusterd : Introduce reset brick) posted (#10) for review on master by Anuradha Talur (atalur)

REVIEW: http://review.gluster.org/12250 (glusterd : Introduce reset brick) posted (#11) for review on master by Anuradha Talur (atalur)

REVIEW: http://review.gluster.org/12250 (glusterd : Introduce reset brick) posted (#12) for review on master by Anuradha Talur (atalur)

REVIEW: http://review.gluster.org/12250 (    glusterd : Introduce reset brick) posted (#13) for review on master by Anuradha Talur (atalur)

REVIEW: http://review.gluster.org/12250 (glusterd : Introduce reset brick) posted (#14) for review on master by Anuradha Talur (atalur)

REVIEW: http://review.gluster.org/12250 (glusterd : Introduce reset brick) posted (#15) for review on master by Anuradha Talur (atalur)

COMMIT: http://review.gluster.org/12250 committed in master by Pranith Kumar Karampuri (pkarampu) 
------
commit 936f8aeac3252951e7fa0cdaa5d260fad3bd5ea0
Author: Anuradha Talur <atalur>
Date:   Mon Aug 22 13:22:03 2016 -0400

    glusterd : Introduce reset brick
    
    The command basically allows replace brick with src and
    dst bricks as same.
    
    Usage:
    gluster v reset-brick <volname> <hostname:brick-path> start
    This command kills the brick to be reset. Once this command is run,
    admin can do other manual operations that they need to do,
    like configuring some options for the brick. Once this is done,
    resetting the brick can be continued with the following options.
    
    gluster v reset-brick <vname> <hostname:brick> <hostname:brick> commit {force}
    
    Does the job of resetting the brick. 'force' option should be used
    when the brick already contains volinfo id.
    
    Problem: On doing a disk-replacement of a brick in a replicate volume
    the following 2 scenarios may occur :
    
    a) there is a chance that reads are served from this replaced-disk brick,
    which leads to empty reads. b) potential data loss if next writes succeed
    only on replaced brick, and heal is done to other bricks from this one.
    
    Solution: After disk-replacement, make sure that reset-brick command is
    run for that brick so that pending markers are set for the brick and it
    is not chosen as source for reads and heal. But, as of now replace-brick
    for the same brick-path is not allowed. In order to fix the above
    mentioned problem, same brick-path replace-brick is needed.
    With this patch reset-brick commit {force} will be allowed even when
    source and destination <hostname:brickpath> are identical as long as
    1) destination brick is not alive
    2) source and destination brick have the same brick uuid and path.
    Also, the destination brick after replace-brick will use the same port
    as the source brick.
    
    Change-Id: I440b9e892ffb781ea4b8563688c3f85c7a7c89de
    BUG: 1266876
    Signed-off-by: Anuradha Talur <atalur>
    Reviewed-on: http://review.gluster.org/12250
    Smoke: Gluster Build System <jenkins.org>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.org>
    Reviewed-by: Ashish Pandey <aspandey>
    Reviewed-by: Pranith Kumar Karampuri <pkarampu>

REVIEW: http://review.gluster.org/15370 (event: fix gf_event messages for replace/reset brick op) posted (#1) for review on master by Anuradha Talur (atalur)

REVIEW: http://review.gluster.org/15370 (event: fix gf_event messages for replace/reset brick op) posted (#2) for review on master by Anuradha Talur (atalur)

REVIEW: http://review.gluster.org/15370 (event: fix gf_event messages for replace/reset brick op) posted (#3) for review on master by Anuradha Talur (atalur)

REVIEW: http://review.gluster.org/15370 (event: fix gf_event messages for replace/reset brick op) posted (#4) for review on master by Anuradha Talur (atalur)

COMMIT: http://review.gluster.org/15370 committed in master by Pranith Kumar Karampuri (pkarampu) 
------
commit 054b09eb5f040ccc0adbb86759da403b49544083
Author: Anuradha Talur <atalur>
Date:   Wed Aug 31 15:27:31 2016 +0530

    event: fix gf_event messages for replace/reset brick op
    
    Change-Id: I80ebeeaffd2b228d7d0796c8d08bc2a051c4ccac
    BUG: 1266876
    Signed-off-by: Anuradha Talur <atalur>
    Reviewed-on: http://review.gluster.org/15370
    Smoke: Gluster Build System <jenkins.org>
    Reviewed-by: Atin Mukherjee <amukherj>
    CentOS-regression: Gluster Build System <jenkins.org>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    Reviewed-by: Pranith Kumar Karampuri <pkarampu>

This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.9.0, please open a new bug report.

glusterfs-3.9.0 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

[1] http://lists.gluster.org/pipermail/gluster-users/2016-November/029281.html
[2] https://www.gluster.org/pipermail/gluster-users/