From de276bd2452302e2e27b94b7f577c25fc351037d Mon Sep 17 00:00:00 2001 From: Avra Sengupta Date: Tue, 19 Jan 2016 16:22:17 +0530 Subject: 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 Reviewed-on: http://review.gluster.org/13260 Smoke: Gluster Build System CentOS-regression: Gluster Build System Reviewed-by: Rajesh Joseph NetBSD-regression: NetBSD Build System Reviewed-by: mohammed rafi kc --- xlators/mgmt/glusterd/src/glusterd-snapshot-utils.c | 3 +++ 1 file changed, 3 insertions(+) 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; -- cgit