From cea409d71083be2758cde0b242eba4341bd4858d Mon Sep 17 00:00:00 2001 From: Gaurav Yadav Date: Thu, 16 Feb 2017 13:02:20 +0530 Subject: glusterd : Fix for error message while removing brick When remove-brick command is issued to a offline brick, glusterd error out the operation with message -: "volume remove-brick start: failed: Found stopped brick :". With this fix while removing brick, error message is modified to "volume remove-brick start: failed: Found stopped brick . Use force option to remove the brick" > Reviewed-on: https://review.gluster.org/16630 > Smoke: Gluster Build System > NetBSD-regression: NetBSD Build System > CentOS-regression: Gluster Build System > Reviewed-by: Atin Mukherjee (cherry picked from commit 64637d040d7f63ef6af6588ae17a557ad0229710) Change-Id: Id40a02fc38cdb526c4629de262967fe2383febe4 BUG: 1438325 Signed-off-by: Gaurav Yadav Reviewed-on: https://review.gluster.org/16979 Smoke: Gluster Build System NetBSD-regression: NetBSD Build System Reviewed-by: Samikshan Bairagya CentOS-regression: Gluster Build System Reviewed-by: Prashanth Pai --- xlators/mgmt/glusterd/src/glusterd-brick-ops.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/xlators/mgmt/glusterd/src/glusterd-brick-ops.c b/xlators/mgmt/glusterd/src/glusterd-brick-ops.c index 43465dbf4d8..475f61e7f39 100644 --- a/xlators/mgmt/glusterd/src/glusterd-brick-ops.c +++ b/xlators/mgmt/glusterd/src/glusterd-brick-ops.c @@ -2040,7 +2040,8 @@ glusterd_remove_brick_validate_bricks (gf1_op_commands cmd, int32_t brick_count, check: if (brickinfo->status != GF_BRICK_STARTED) { snprintf (msg, sizeof (msg), "Found stopped " - "brick %s", brick); + "brick %s. Use force option to " + "remove the offline brick" , brick); *errstr = gf_strdup (msg); ret = -1; goto out; -- cgit