diff options
| author | Raghavendra Bhat <raghavendra@redhat.com> | 2014-05-27 17:46:08 +0530 |
|---|---|---|
| committer | Krishnan Parthasarathi <kparthas@redhat.com> | 2014-06-01 22:26:25 -0700 |
| commit | c50766a76783e15b25a9bc57a4ece190df4e2a5b (patch) | |
| tree | f528d2838e129055abf2fd71f0bf3d8a31a2a24f | |
| parent | 58c4a882c37eb2042cbea341fceb4bd0ef028d9a (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.c | 3 |
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", |