diff options
| author | Atin Mukherjee <amukherj@redhat.com> | 2017-09-07 19:14:23 +0530 |
|---|---|---|
| committer | Atin Mukherjee <amukherj@redhat.com> | 2017-09-19 02:54:10 +0000 |
| commit | 7f70d38b66ce755f848ff0197814457a28b321df (patch) | |
| tree | 1405b386f301715953a18c7050e1a29487c9f115 /tests/bugs/glusterd/bug-1483058-replace-brick-quorum-validation.t | |
| parent | f6775423d7fbe71fe35e6154388e77aae2e264ad (diff) | |
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@redhat.com>
Reviewed-on: https://review.gluster.org/18226
Smoke: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: N Balachandran <nbalacha@redhat.com>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Diffstat (limited to 'tests/bugs/glusterd/bug-1483058-replace-brick-quorum-validation.t')
| -rw-r--r-- | tests/bugs/glusterd/bug-1483058-replace-brick-quorum-validation.t | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/bugs/glusterd/bug-1483058-replace-brick-quorum-validation.t b/tests/bugs/glusterd/bug-1483058-replace-brick-quorum-validation.t index 28b2fbbb9cf..3dbe28a7917 100644 --- a/tests/bugs/glusterd/bug-1483058-replace-brick-quorum-validation.t +++ b/tests/bugs/glusterd/bug-1483058-replace-brick-quorum-validation.t @@ -18,7 +18,7 @@ TEST $CLI_1 peer probe $H3; EXPECT_WITHIN $PROBE_TIMEOUT 2 peer_count # Lets create the volume and set quorum type as a server -TEST $CLI_1 volume create $V0 $H1:$B1/${V0}0 $H2:$B2/${V0}1 $H3:$B3/${V0}2 +TEST $CLI_1 volume create $V0 replica 3 $H1:$B1/${V0}0 $H2:$B2/${V0}1 $H3:$B3/${V0}2 TEST $CLI_1 volume set $V0 cluster.server-quorum-type server # Start the volume |