From 0fa39220603233dc0030eeedfc7f05947c35da7c Mon Sep 17 00:00:00 2001 From: Sunny Kumar Date: Mon, 13 Nov 2017 13:47:53 +0530 Subject: snapshot : snapshot creation failed after brick reset/replace Problem : snapshot creation was failing after brick reset/replace Fix : changed code to set mount_dir value in rsp_dict during prerequisites phase i.e glusterd_brick_op_prerequisites call and removed form prevalidate phase. Signed-off-by: Sunny Kumar Change-Id: Ief5d0fafe882a7eb1a7da8535b7c7ce6f011604c BUG: 1512451 --- xlators/mgmt/glusterd/src/glusterd-replace-brick.c | 8 -------- xlators/mgmt/glusterd/src/glusterd-reset-brick.c | 10 ---------- xlators/mgmt/glusterd/src/glusterd-utils.c | 9 +++++++++ 3 files changed, 9 insertions(+), 18 deletions(-) (limited to 'xlators') diff --git a/xlators/mgmt/glusterd/src/glusterd-replace-brick.c b/xlators/mgmt/glusterd/src/glusterd-replace-brick.c index 08a6df0235f..f282c328387 100644 --- a/xlators/mgmt/glusterd/src/glusterd-replace-brick.c +++ b/xlators/mgmt/glusterd/src/glusterd-replace-brick.c @@ -319,14 +319,6 @@ glusterd_op_stage_replace_brick (dict_t *dict, char **op_errstr, goto out; } - ret = dict_set_dynstr_with_alloc (rsp_dict, "brick1.mount_dir", - dst_brickinfo->mount_dir); - if (ret) { - gf_msg (this->name, GF_LOG_ERROR, 0, - GD_MSG_DICT_SET_FAILED, - "Failed to set brick1.mount_dir"); - goto out; - } ret = dict_set_int32 (rsp_dict, "brick_count", 1); if (ret) { diff --git a/xlators/mgmt/glusterd/src/glusterd-reset-brick.c b/xlators/mgmt/glusterd/src/glusterd-reset-brick.c index abb44e01df7..10ee6f4ef62 100644 --- a/xlators/mgmt/glusterd/src/glusterd-reset-brick.c +++ b/xlators/mgmt/glusterd/src/glusterd-reset-brick.c @@ -208,16 +208,6 @@ glusterd_reset_brick_prevalidate (dict_t *dict, char **op_errstr, goto out; } - ret = dict_set_dynstr_with_alloc (rsp_dict, - "brick1.mount_dir", - dst_brickinfo->mount_dir); - if (ret) { - gf_msg (this->name, GF_LOG_ERROR, 0, - GD_MSG_DICT_SET_FAILED, - "Failed to set brick1.mount_dir"); - goto out; - } - ret = dict_set_int32 (rsp_dict, "brick_count", 1); if (ret) { gf_msg (this->name, GF_LOG_ERROR, 0, diff --git a/xlators/mgmt/glusterd/src/glusterd-utils.c b/xlators/mgmt/glusterd/src/glusterd-utils.c index a50a294176b..fe96d6be094 100644 --- a/xlators/mgmt/glusterd/src/glusterd-utils.c +++ b/xlators/mgmt/glusterd/src/glusterd-utils.c @@ -13884,6 +13884,15 @@ glusterd_brick_op_prerequisites (dict_t *dict, (*src_brickinfo)->port); } } + /* setting mount_dir */ + ret = dict_set_dynstr_with_alloc (rsp_dict, "brick1.mount_dir", + (*src_brickinfo)->mount_dir); + if (ret) { + gf_msg (this->name, GF_LOG_ERROR, 0, + GD_MSG_DICT_SET_FAILED, + "Failed to set brick1.mount_dir"); + goto out; + } v = *volinfo; b = *src_brickinfo; -- cgit