diff options
| author | Avra Sengupta <asengupt@redhat.com> | 2016-01-19 16:22:17 +0530 |
|---|---|---|
| committer | Rajesh Joseph <rjoseph@redhat.com> | 2016-01-19 22:57:54 -0800 |
| commit | de276bd2452302e2e27b94b7f577c25fc351037d (patch) | |
| tree | 3dee35dae47a198630001f2a557ab9696761bedd /xlators/mgmt | |
| parent | 67b7752eccd8eae6a91f3bc3ff55d65dcf858dc7 (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.c | 3 |
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; |