summaryrefslogtreecommitdiffstats
path: root/xlators
diff options
context:
space:
mode:
authorhari <hgowtham@redhat.com>2016-05-23 12:07:21 +0530
committerJeff Darcy <jdarcy@redhat.com>2016-07-15 06:46:10 -0700
commitc28aca72289c6526abc48d08f669c50e13ef4ab5 (patch)
treec4af12cc1d21c413e6327d8dfe66d9c25e381e90 /xlators
parentce1905c58fd604f1ef00a55609a832d1cf658c7e (diff)
Glusterd: printing the node details on error message of rebalance
back-port of : http://review.gluster.org/#/c/14495 Problem: on the rebalance start with one of the glusterd being down among the volume, the error message says only about the brick path. Fix: adding the node details >Change-Id: I5827d3a9a15b0461c9ce3a51c0b16246ca58f335 >BUG: 1337899 >Signed-off-by: hari <hgowtham@redhat.com> Change-Id: I3075f3a73e289dfe577742a3d5086531026f567d BUG: 1339923 Signed-off-by: hari gowtham <hgowtham@redhat.com> Reviewed-on: http://review.gluster.org/14540 NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> CentOS-regression: Gluster Build System <jenkins@build.gluster.org> Tested-by: hari gowtham <hari.gowtham005@gmail.com> Reviewed-by: Zhou Zhengping <johnzzpcrystal@gmail.com> Smoke: Gluster Build System <jenkins@build.gluster.org> Reviewed-by: Jeff Darcy <jdarcy@redhat.com>
Diffstat (limited to 'xlators')
-rw-r--r--xlators/mgmt/glusterd/src/glusterd-rebalance.c12
1 files changed, 7 insertions, 5 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-rebalance.c b/xlators/mgmt/glusterd/src/glusterd-rebalance.c
index 9065c6c..2781b5d 100644
--- a/xlators/mgmt/glusterd/src/glusterd-rebalance.c
+++ b/xlators/mgmt/glusterd/src/glusterd-rebalance.c
@@ -722,16 +722,18 @@ glusterd_op_stage_rebalance (dict_t *dict, char **op_errstr)
peerinfo = glusterd_peerinfo_find_by_uuid
(brickinfo->uuid);
if (!peerinfo) {
- gf_asprintf (op_errstr, "Host node of "
- "brick %s doesn't belong "
- "to cluster",
+ gf_asprintf (op_errstr, "Host node %s "
+ "of brick %s doesn't "
+ "belong to cluster",
+ brickinfo->hostname,
brickinfo->path);
ret = -1;
rcu_read_unlock ();
goto out;
} else if (!peerinfo->connected) {
- gf_asprintf (op_errstr, "Host node of "
- "brick %s is down",
+ gf_asprintf (op_errstr, "Host node %s "
+ "of brick %s is down",
+ brickinfo->hostname,
brickinfo->path);
ret = -1;
rcu_read_unlock ();