andy/glusterfs
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

glusterd: add-brick failed to start bricks when more than one were given

Browse Source 
Change-Id: I1487487a656e15e1a9a8685524ea40f0636e0e50
BUG: 864772
Signed-off-by: Krishnan Parthasarathi <kparthas@redhat.com>
Reviewed-on: http://review.gluster.org/4051
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Kaushal M <kaushal@redhat.com>
Reviewed-by: Anand Avati <avati@redhat.com>
This commit is contained in:
Krishnan Parthasarathi 2012-10-10 10:23:37 +05:30 committed by Anand Avati
parent 7ed127844d
commit 751da4675c
1 changed files with 16 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
xlators/mgmt/glusterd/src/glusterd-brick-ops.c
View File

@ -916,6 +916,14 @@ glusterd_op_perform_add_bricks (glusterd_volinfo_t *volinfo, int32_t count,
volinfo->sub_count = ((volinfo->dist_leaf_count == 1) ? 0:
volinfo->dist_leaf_count);
ret = glusterd_create_volfiles_and_notify_services (volinfo);
if (ret)
goto out;
ret = 0;
if (GLUSTERD_STATUS_STARTED != volinfo->status)
goto out;
brick_list = gf_strdup (bricks);
free_ptr2 = brick_list;
i = 1;
@ -923,17 +931,16 @@ glusterd_op_perform_add_bricks (glusterd_volinfo_t *volinfo, int32_t count,
if (count)
brick = strtok_r (brick_list+1, " \n", &saveptr);
ret = glusterd_create_volfiles_and_notify_services (volinfo);
if (ret)
goto out;
while (i <= count) {
if (GLUSTERD_STATUS_STARTED == volinfo->status) {
ret = glusterd_brick_start (volinfo, brickinfo);
if (ret)
goto out;
}
ret = glusterd_volume_brickinfo_get_by_brick (brick, volinfo,
&brickinfo);
if (ret)
goto out;
ret = glusterd_brick_start (volinfo, brickinfo);
if (ret)
goto out;
i++;
brick = strtok_r (NULL, " \n", &saveptr);
}