summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRaghavendra Bhat <raghavendrabhat@gluster.com>2011-05-02 05:36:17 +0000
committerAnand Avati <avati@gluster.com>2011-05-11 19:18:12 -0700
commit4ba7635f4d3f199dbb082b45cfe7b49f7e243587 (patch)
tree812fb87e17337a33d8da048145ca6406cb70dc84
parent276c0cecadbe763e98c5cfd18b3ad32822964789 (diff)
glusterd: send the duplicate brick info to the cli
Signed-off-by: Raghavendra Bhat <raghavendrabhat@gluster.com> Signed-off-by: Anand Avati <avati@gluster.com> BUG: 2803 ([glusterfs-3.2.0qa14]: volume create succeeds even though there is a volume with an export directory in the path of current volume) URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=2803
-rw-r--r--xlators/mgmt/glusterd/src/glusterd-handler.c10
-rw-r--r--xlators/mgmt/glusterd/src/glusterd-utils.c2
2 files changed, 6 insertions, 6 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-handler.c b/xlators/mgmt/glusterd/src/glusterd-handler.c
index bf601efcd..df7a5f9fc 100644
--- a/xlators/mgmt/glusterd/src/glusterd-handler.c
+++ b/xlators/mgmt/glusterd/src/glusterd-handler.c
@@ -1001,9 +1001,8 @@ glusterd_handle_create_volume (rpcsvc_request_t *req)
if (!ret) {
ret = -1;
snprintf (err_str, sizeof (err_str), "Brick: %s:%s, %s"
- " in the arguments mean the same",
- tmpbrkinfo->hostname, tmpbrkinfo->path,
- brick);
+ " one of the arguments contain the other",
+ tmpbrkinfo->hostname, tmpbrkinfo->path, brick);
goto out;
}
list_add_tail (&brickinfo->brick_list, &tmpvolinfo.bricks);
@@ -1423,9 +1422,8 @@ brick_val:
if (!ret) {
ret = -1;
snprintf (err_str, sizeof (err_str), "Brick: %s:%s, %s"
- " in the arguments mean the same",
- tmpbrkinfo->hostname, tmpbrkinfo->path,
- brick);
+ " one of the arguments contain the other",
+ tmpbrkinfo->hostname, tmpbrkinfo->path, brick);
goto out;
}
list_add_tail (&brickinfo->brick_list, &tmpvolinfo.bricks);
diff --git a/xlators/mgmt/glusterd/src/glusterd-utils.c b/xlators/mgmt/glusterd/src/glusterd-utils.c
index ec3449de4..c1c9481e1 100644
--- a/xlators/mgmt/glusterd/src/glusterd-utils.c
+++ b/xlators/mgmt/glusterd/src/glusterd-utils.c
@@ -739,6 +739,7 @@ glusterd_volume_brickinfo_get (uuid_t uuid, char *hostname, char *path,
"given path %s lies"
" within %s", path,
brickiter->path);
+ *brickinfo = brickiter;
break;
}
} else
@@ -748,6 +749,7 @@ glusterd_volume_brickinfo_get (uuid_t uuid, char *hostname, char *path,
" %s", brickiter->path,
path);
ret = 0;
+ *brickinfo = brickiter;
break;
}
}