diff options
| author | Avra Sengupta <asengupt@redhat.com> | 2016-01-19 16:22:17 +0530 |
|---|---|---|
| committer | Pranith Kumar Karampuri <pkarampu@redhat.com> | 2016-01-19 22:57:21 -0800 |
| commit | b273915811fc484698a46f36597581dd0a5753e7 (patch) | |
| tree | 765f3d1140bf60f51c6b6fcc08ee40d6bfd07d04 | |
| parent | 11d91f4e5fb66596addd8906b2f65a4137bd580a (diff) | |
snapshot: Return before redundant quorum check
Backport of http://review.gluster.org/#/c/13260/
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.
Change-Id: I35661d05a15be0109aaae51b4fe0d5a8ca4333ad
BUG: 1299822
Signed-off-by: Avra Sengupta <asengupt@redhat.com>
Reviewed-on: http://review.gluster.org/13261
Smoke: Gluster Build System <jenkins@build.gluster.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Rajesh Joseph <rjoseph@redhat.com>
| -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 51397e4138c..58dce29f613 100644 --- a/xlators/mgmt/glusterd/src/glusterd-snapshot-utils.c +++ b/xlators/mgmt/glusterd/src/glusterd-snapshot-utils.c @@ -2753,6 +2753,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; |