summaryrefslogtreecommitdiffstats
path: root/xlators/mgmt
diff options
context:
space:
mode:
authorAmar Tumballi <amar@gluster.com>2011-09-26 20:43:17 +0530
committerVijay Bellur <vijay@gluster.com>2011-09-27 05:35:03 -0700
commit68ff38bd2f77c62cd0c5729b047003694a7d63af (patch)
treed7c0f052f08dc0da1eb01f127405c7e1b090dc71 /xlators/mgmt
parentb894944857a7ce61a3bd31566d6169ed8d19ce8b (diff)
glusted: while doing add-brick, handle distribute volume separately
so the logic of different type of counts etc won't come into action, which would fail the add-brick operation. Change-Id: I5c8769d6d1c2433ffb084e5dce9c1e01678cff40 BUG: 3630 Reviewed-on: http://review.gluster.com/505 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Krishnan Parthasarathi <kp@gluster.com>
Diffstat (limited to 'xlators/mgmt')
-rw-r--r--xlators/mgmt/glusterd/src/glusterd-brick-ops.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-brick-ops.c b/xlators/mgmt/glusterd/src/glusterd-brick-ops.c
index f488996c04d..75087d54771 100644
--- a/xlators/mgmt/glusterd/src/glusterd-brick-ops.c
+++ b/xlators/mgmt/glusterd/src/glusterd-brick-ops.c
@@ -127,6 +127,9 @@ glusterd_handle_add_brick (rpcsvc_request_t *req)
}
+ if (volinfo->type == GF_CLUSTER_TYPE_NONE)
+ goto brick_val;
+
/* If any of this is true, some thing is wrong */
if (!brick_count || !volinfo->sub_count) {
ret = -1;
@@ -135,9 +138,6 @@ glusterd_handle_add_brick (rpcsvc_request_t *req)
goto out;
}
- if (volinfo->type == GF_CLUSTER_TYPE_NONE)
- goto brick_val;
-
total_bricks = volinfo->brick_count + brick_count;
/* If the brick count is less than dist_leaf_count then, allow
add-brick only for plain replicate volume since in plain stripe