diff options
author | Raghavendra Bhat <raghavendrabhat@gluster.com> | 2011-10-13 17:29:34 +0530 |
---|---|---|
committer | Vijay Bellur <vijay@gluster.com> | 2011-10-17 01:49:57 -0700 |
commit | 7f8d2e19bb09cb49a19f835bd69b03612a1f007e (patch) | |
tree | 2571da8e88cd101fc64bc0cbe9c89d73cc9922b4 /xlators/mgmt/glusterd/src/glusterd-handler.c | |
parent | 90d691d049420f712f5bd1898c75cc66edd1b315 (diff) |
mgmt/glusterd: send the duplicate brick info to the cli
suppose one gives create volume from a peer whose bricks are
hostname:/a/b hostname:/a/b/c, then send that error information
to the cli.
Change-Id: Ic2149647f753fcf9218e468f13c52d0fc013a5d3
BUG: 2803
Reviewed-on: http://review.gluster.com/582
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijay Bellur <vijay@gluster.com>
Diffstat (limited to 'xlators/mgmt/glusterd/src/glusterd-handler.c')
-rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-handler.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-handler.c b/xlators/mgmt/glusterd/src/glusterd-handler.c index a4af71e3bc1..552998ad058 100644 --- a/xlators/mgmt/glusterd/src/glusterd-handler.c +++ b/xlators/mgmt/glusterd/src/glusterd-handler.c @@ -1000,8 +1000,9 @@ glusterd_handle_create_volume (rpcsvc_request_t *req) &tmpvolinfo, &tmpbrkinfo); if (!ret) { ret = -1; - snprintf (err_str, sizeof (err_str), "Brick: %s:%s, %s" - " in the arguments mean the same", + snprintf (err_str, sizeof (err_str), "Brick: %s:%s and " + "Brick: %s have overlapping export directories" + " from the same peer", tmpbrkinfo->hostname, tmpbrkinfo->path, brick); goto out; @@ -1422,8 +1423,9 @@ brick_val: &tmpvolinfo, &tmpbrkinfo); if (!ret) { ret = -1; - snprintf (err_str, sizeof (err_str), "Brick: %s:%s, %s" - " in the arguments mean the same", + snprintf (err_str, sizeof (err_str), "Brick: %s:%s and " + "Brick: %s have overlapping export directories" + " from the same peer", tmpbrkinfo->hostname, tmpbrkinfo->path, brick); goto out; |