|
 |
50dc83 |
From d08083d057d6cc7136128cad6ecefba43b886c4c Mon Sep 17 00:00:00 2001
|
|
|
50dc83 |
From: Vishal Pandey <vpandey@redhat.com>
|
|
|
50dc83 |
Date: Thu, 9 May 2019 14:37:22 +0530
|
|
|
50dc83 |
Subject: [PATCH 138/141] glusterd: Add gluster volume stop operation to
|
|
|
50dc83 |
glusterd_validate_quorum()
|
|
|
50dc83 |
|
|
|
50dc83 |
ISSUE: gluster volume stop succeeds even if quorum is not met.
|
|
|
50dc83 |
|
|
|
50dc83 |
Fix: Add GD_OP_STOP_VOLUME to gluster_validate_quorum in
|
|
|
50dc83 |
glusterd_mgmt_v3_pre_validate ().
|
|
|
50dc83 |
|
|
|
50dc83 |
Since the volume stop command has been ported from synctask to mgmt_v3,
|
|
|
50dc83 |
the quorum check was missed out.
|
|
|
50dc83 |
|
|
|
50dc83 |
>upstream patch : https://review.gluster.org/#/c/glusterfs/+/22692/
|
|
|
50dc83 |
|
|
|
50dc83 |
>Change-Id: I7a634ad89ec2e286ea262d7952061efad5360042
|
|
|
50dc83 |
>fixes: bz#1690753
|
|
|
50dc83 |
>Signed-off-by: Vishal Pandey <vpandey@redhat.com>
|
|
|
50dc83 |
|
|
|
50dc83 |
BUG: 1706893
|
|
|
50dc83 |
Change-Id: I7a634ad89ec2e286ea262d7952061efad5360042
|
|
|
50dc83 |
Signed-off-by: Vishal Pandey <vpandey@redhat.com>
|
|
|
50dc83 |
Reviewed-on: https://code.engineering.redhat.com/gerrit/169949
|
|
|
50dc83 |
Tested-by: RHGS Build Bot <nigelb@redhat.com>
|
|
|
50dc83 |
Reviewed-by: Sunil Kumar Heggodu Gopala Acharya <sheggodu@redhat.com>
|
|
|
50dc83 |
---
|
|
|
50dc83 |
tests/bugs/glusterd/quorum-validation.t | 4 +++-
|
|
|
50dc83 |
xlators/mgmt/glusterd/src/glusterd-mgmt.c | 2 +-
|
|
|
50dc83 |
2 files changed, 4 insertions(+), 2 deletions(-)
|
|
|
50dc83 |
|
|
|
50dc83 |
diff --git a/tests/bugs/glusterd/quorum-validation.t b/tests/bugs/glusterd/quorum-validation.t
|
|
|
50dc83 |
index 05aef4e..ff46729 100644
|
|
|
50dc83 |
--- a/tests/bugs/glusterd/quorum-validation.t
|
|
|
50dc83 |
+++ b/tests/bugs/glusterd/quorum-validation.t
|
|
|
50dc83 |
@@ -34,9 +34,11 @@ TEST ! $CLI_1 volume add-brick $V0 $H1:$B1/${V0}2
|
|
|
50dc83 |
TEST ! $CLI_1 volume remove-brick $V0 $H1:$B1/${V0}0 start
|
|
|
50dc83 |
TEST ! $CLI_1 volume set $V0 barrier enable
|
|
|
50dc83 |
|
|
|
50dc83 |
-# Now execute a command which goes through op state machine and it should fail
|
|
|
50dc83 |
TEST ! $CLI_1 volume profile $V0 start
|
|
|
50dc83 |
|
|
|
50dc83 |
+#bug-1690753 - Volume stop when quorum not met is successful
|
|
|
50dc83 |
+TEST ! $CLI_1 volume stop $V0
|
|
|
50dc83 |
+
|
|
|
50dc83 |
#Bring back the 2nd glusterd
|
|
|
50dc83 |
TEST $glusterd_2
|
|
|
50dc83 |
|
|
|
50dc83 |
diff --git a/xlators/mgmt/glusterd/src/glusterd-mgmt.c b/xlators/mgmt/glusterd/src/glusterd-mgmt.c
|
|
|
50dc83 |
index 61ad66e..ec78913 100644
|
|
|
50dc83 |
--- a/xlators/mgmt/glusterd/src/glusterd-mgmt.c
|
|
|
50dc83 |
+++ b/xlators/mgmt/glusterd/src/glusterd-mgmt.c
|
|
|
50dc83 |
@@ -1059,7 +1059,7 @@ glusterd_mgmt_v3_pre_validate(glusterd_op_t op, dict_t *req_dict,
|
|
|
50dc83 |
goto out;
|
|
|
50dc83 |
}
|
|
|
50dc83 |
|
|
|
50dc83 |
- if (op == GD_OP_PROFILE_VOLUME) {
|
|
|
50dc83 |
+ if (op == GD_OP_PROFILE_VOLUME || op == GD_OP_STOP_VOLUME) {
|
|
|
50dc83 |
ret = glusterd_validate_quorum(this, op, req_dict, op_errstr);
|
|
|
50dc83 |
if (ret) {
|
|
|
50dc83 |
gf_msg(this->name, GF_LOG_ERROR, 0, GD_MSG_SERVER_QUORUM_NOT_MET,
|
|
|
50dc83 |
--
|
|
|
50dc83 |
1.8.3.1
|
|
|
50dc83 |
|