summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRaghavendra Bhat <raghavendra@redhat.com>2014-05-27 17:46:08 +0530
committerKrishnan Parthasarathi <kparthas@redhat.com>2014-06-01 22:26:25 -0700
commitc50766a76783e15b25a9bc57a4ece190df4e2a5b (patch)
treef528d2838e129055abf2fd71f0bf3d8a31a2a24f
parent58c4a882c37eb2042cbea341fceb4bd0ef028d9a (diff)
mgmt/glusterd: return failure when server quorum is not met
Change-Id: I0a7df141d01cf70da5aac9658b2a5a19f660dd3b BUG: 1101561 Signed-off-by: Raghavendra Bhat <raghavendra@redhat.com> Reviewed-on: http://review.gluster.org/7894 Reviewed-by: Sachin Pandit <spandit@redhat.com> Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Krishnan Parthasarathi <kparthas@redhat.com> Tested-by: Krishnan Parthasarathi <kparthas@redhat.com>
-rw-r--r--xlators/mgmt/glusterd/src/glusterd-utils.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-utils.c b/xlators/mgmt/glusterd/src/glusterd-utils.c
index ced41fb3a55..bd9f7ba39af 100644
--- a/xlators/mgmt/glusterd/src/glusterd-utils.c
+++ b/xlators/mgmt/glusterd/src/glusterd-utils.c
@@ -12759,7 +12759,7 @@ int32_t
glusterd_snap_quorum_check (dict_t *dict, gf_boolean_t snap_volume,
char **op_errstr)
{
- int32_t ret = 0;
+ int32_t ret = -1;
xlator_t *this = NULL;
int32_t snap_command = 0;
char err_str[PATH_MAX] = {0, };
@@ -12793,6 +12793,7 @@ glusterd_snap_quorum_check (dict_t *dict, gf_boolean_t snap_volume,
case GF_SNAP_OPTION_TYPE_DELETE:
case GF_SNAP_OPTION_TYPE_RESTORE:
if (!does_gd_meet_server_quorum (this)) {
+ ret = -1;
snprintf (err_str, sizeof (err_str),
"glusterds are not in quorum");
gf_log (this->name, GF_LOG_WARNING, "%s",