diff options
| author | hari <hgowtham@redhat.com> | 2016-05-23 12:07:21 +0530 |
|---|---|---|
| committer | Atin Mukherjee <amukherj@redhat.com> | 2016-05-25 08:12:41 -0700 |
| commit | fb57ac1acb228301edb98f2fe7cf93471363eb4a (patch) | |
| tree | 1dc4b4b4a48f1613aae568a51997a9e8d5ba02f6 /xlators/mgmt/glusterd/src | |
| parent | edcc6a4ef4f39d5e3073195d45011e3cdfa7c091 (diff) | |
Glusterd: printing the node details on error message of rebalance
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>
Reviewed-on: http://review.gluster.org/14495
Tested-by: hari gowtham <hari.gowtham005@gmail.com>
Smoke: Gluster Build System <jenkins@build.gluster.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
Diffstat (limited to 'xlators/mgmt/glusterd/src')
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-rebalance.c | 12 |
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 773e59b769a..e0eee02ed52 100644 --- a/xlators/mgmt/glusterd/src/glusterd-rebalance.c +++ b/xlators/mgmt/glusterd/src/glusterd-rebalance.c @@ -717,16 +717,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 (); |