[Bugs] [Bug 1493975] New: disallow replace brick operation on plain distribute volume

[bugzilla at redhat.com]

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

            Bug ID: 1493975
           Summary: disallow replace brick operation on plain distribute
                    volume
           Product: GlusterFS
           Version: 3.12
         Component: glusterd
          Keywords: Triaged
          Assignee: bugs at gluster.org
          Reporter: amukherj at redhat.com
                CC: bugs at gluster.org
        Depends On: 1489432



+++ This bug was initially created as a clone of Bug #1489432 +++

Description of problem:

disallow replace brick operation on plain distribute volume as it would lead to
a data loss.

Version-Release number of selected component (if applicable):
mainline

--- Additional comment from Worker Ant on 2017-09-07 10:10:45 EDT ---

REVIEW: https://review.gluster.org/18226 (glusterd: disallow replace brick for
dist only volumes) posted (#1) for review on master by Atin Mukherjee
(amukherj at redhat.com)

--- Additional comment from Worker Ant on 2017-09-08 03:09:03 EDT ---

REVIEW: https://review.gluster.org/18226 (glusterd: disallow replace brick for
dist only volumes) posted (#2) for review on master by Atin Mukherjee
(amukherj at redhat.com)

--- Additional comment from Worker Ant on 2017-09-18 09:22:20 EDT ---

REVIEW: https://review.gluster.org/18226 (glusterd: disallow replace brick for
dist only volumes) posted (#3) for review on master by Atin Mukherjee
(amukherj at redhat.com)

--- Additional comment from Worker Ant on 2017-09-18 22:54:14 EDT ---

COMMIT: https://review.gluster.org/18226 committed in master by Atin Mukherjee
(amukherj at redhat.com) 
------
commit 7f70d38b66ce755f848ff0197814457a28b321df
Author: Atin Mukherjee <amukherj at redhat.com>
Date:   Thu Sep 7 19:14:23 2017 +0530

    glusterd: disallow replace brick for dist only volumes

    Allowing replace-brick on dist only volumes will lead to data loss. This
    patch blocks replace brick commit force to fail if a volume is dist
    only.

    Also removing tests/basic/pump.t as its of no use as per the discussion
    in
    http://lists.gluster.org/pipermail/gluster-devel/2017-September/053652.html

    Change-Id: Iabb0c16f865f3fc361b64a19bfcf0c0fbb5c2682
    BUG: 1489432
    Signed-off-by: Atin Mukherjee <amukherj at redhat.com>
    Reviewed-on: https://review.gluster.org/18226
    Smoke: Gluster Build System <jenkins at build.gluster.org>
    Reviewed-by: N Balachandran <nbalacha at redhat.com>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.org>


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1489432
[Bug 1489432] disallow replace brick operation on plain distribute volume
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.