summaryrefslogtreecommitdiffstats
path: root/xlators
diff options
context:
space:
mode:
authorAtin Mukherjee <amukherj@redhat.com>2017-06-27 21:09:49 +0530
committerAtin Mukherjee <amukherj@redhat.com>2017-06-28 06:19:01 +0000
commit24d09edf4b13d72a8707c801939921de0d32d4dd (patch)
tree58d3fde1d64bd3fc476b8af1d7de1b8765fbfc8f /xlators
parent68cb9356d19cba5da7f8a719baca47d34e1740b7 (diff)
glusterd: mark brickinfo to started on successful attach
brickinfo's port & status should be filled up only when attach brick is successful. Change-Id: I68b181be37cb94d176f0f4692e8d9dac5493181c BUG: 1465559 Signed-off-by: Atin Mukherjee <amukherj@redhat.com> Reviewed-on: https://review.gluster.org/17640 Reviewed-by: Jeff Darcy <jeff@pl.atyp.us> Smoke: Gluster Build System <jenkins@build.gluster.org> Reviewed-by: Samikshan Bairagya <samikshan@gmail.com> CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Diffstat (limited to 'xlators')
-rw-r--r--xlators/mgmt/glusterd/src/glusterd-utils.c9
1 files changed, 4 insertions, 5 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-utils.c b/xlators/mgmt/glusterd/src/glusterd-utils.c
index f0c12315160..7e50d597fc2 100644
--- a/xlators/mgmt/glusterd/src/glusterd-utils.c
+++ b/xlators/mgmt/glusterd/src/glusterd-utils.c
@@ -5076,11 +5076,6 @@ attach_brick (xlator_t *this,
"adding brick to process failed");
return -1;
}
-
- brickinfo->port = other_brick->port;
- brickinfo->status = GF_BRICK_STARTED;
- brickinfo->rpc = rpc_clnt_ref (other_brick->rpc);
-
GLUSTERD_GET_BRICK_PIDFILE (pidfile1, other_vol, other_brick, conf);
GLUSTERD_GET_BRICK_PIDFILE (pidfile2, volinfo, brickinfo, conf);
@@ -5106,6 +5101,10 @@ attach_brick (xlator_t *this,
successfully
*/
glusterd_copy_file (pidfile1, pidfile2);
+ brickinfo->port = other_brick->port;
+ brickinfo->status = GF_BRICK_STARTED;
+ brickinfo->rpc =
+ rpc_clnt_ref (other_brick->rpc);
return 0;
}
}