summaryrefslogtreecommitdiffstats
path: root/xlators/mgmt
diff options
context:
space:
mode:
authorAvra Sengupta <asengupt@redhat.com>2016-01-19 16:22:17 +0530
committerRajesh Joseph <rjoseph@redhat.com>2016-01-19 22:57:54 -0800
commitde276bd2452302e2e27b94b7f577c25fc351037d (patch)
tree3dee35dae47a198630001f2a557ab9696761bedd /xlators/mgmt
parent67b7752eccd8eae6a91f3bc3ff55d65dcf858dc7 (diff)
snapshot: Return before redundant quorum check
As of today, we don't support creation of snapshot even if one brick is down. Hence the older quorum check is redundant. Returning after performing the check to see if all bricks are up. BUG: 1299819 Change-Id: I35661d05a15be0109aaae51b4fe0d5a8ca4333ad Signed-off-by: Avra Sengupta <asengupt@redhat.com> Reviewed-on: http://review.gluster.org/13260 Smoke: Gluster Build System <jenkins@build.gluster.com> CentOS-regression: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Rajesh Joseph <rjoseph@redhat.com> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> Reviewed-by: mohammed rafi kc <rkavunga@redhat.com>
Diffstat (limited to 'xlators/mgmt')
-rw-r--r--xlators/mgmt/glusterd/src/glusterd-snapshot-utils.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-snapshot-utils.c b/xlators/mgmt/glusterd/src/glusterd-snapshot-utils.c
index 973e44c3447..1853028102c 100644
--- a/xlators/mgmt/glusterd/src/glusterd-snapshot-utils.c
+++ b/xlators/mgmt/glusterd/src/glusterd-snapshot-utils.c
@@ -2750,6 +2750,9 @@ glusterd_volume_quorum_calculate (glusterd_volinfo_t *volinfo, dict_t *dict,
*op_errstr = gf_strdup (err_str);
*op_errno = EG_BRCKDWN;
goto out;
+ } else {
+ quorum_met = _gf_true;
+ goto out;
}
up_count = volinfo->dist_leaf_count - down_count;