From 9fe52c92ace1c8c1e6e016aad8bb14973d6c9dc0 Mon Sep 17 00:00:00 2001 From: Amar Tumballi Date: Thu, 26 Aug 2010 05:53:41 +0000 Subject: glusterd: handle failure of 'getspec' in case of '-' in volume name Signed-off-by: Amar Tumballi Signed-off-by: Anand V. Avati BUG: 1433 () URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=1433 --- extras/volgen/CreateVolfile.py | 2 +- xlators/mgmt/glusterd/src/glusterd-handshake.c | 5 +---- xlators/mgmt/glusterd/src/glusterd-utils.c | 4 ++-- 3 files changed, 4 insertions(+), 7 deletions(-) diff --git a/extras/volgen/CreateVolfile.py b/extras/volgen/CreateVolfile.py index 22cffcffe29..c8143a03aba 100644 --- a/extras/volgen/CreateVolfile.py +++ b/extras/volgen/CreateVolfile.py @@ -348,7 +348,7 @@ class CreateVolfile: i = 0 for export in self.host_dict[self.host]: export_volfile = "%s/%s-%d.vol" % (self.conf_dir, - str(self.host + '-' + self.volume_name), + str(self.volume_name + '.' + self.host), i) i += 1 exp_fd = file ("%s" % (export_volfile),"w") diff --git a/xlators/mgmt/glusterd/src/glusterd-handshake.c b/xlators/mgmt/glusterd/src/glusterd-handshake.c index 43a93731790..471fed3f1ce 100644 --- a/xlators/mgmt/glusterd/src/glusterd-handshake.c +++ b/xlators/mgmt/glusterd/src/glusterd-handshake.c @@ -56,10 +56,7 @@ build_volfile_path (const char *volname, char *path, ret = glusterd_volinfo_find (dup_volname, &volinfo); if (ret) { /* Split the volume name */ - vol = strtok_r (dup_volname, "-", &tmp); - if (!vol) - goto out; - vol = strtok_r (NULL, "-", &tmp); + vol = strtok_r (dup_volname, ".", &tmp); if (!vol) goto out; ret = glusterd_volinfo_find (vol, &volinfo); diff --git a/xlators/mgmt/glusterd/src/glusterd-utils.c b/xlators/mgmt/glusterd/src/glusterd-utils.c index 2132c56ff39..e8d9b4634c2 100644 --- a/xlators/mgmt/glusterd/src/glusterd-utils.c +++ b/xlators/mgmt/glusterd/src/glusterd-utils.c @@ -762,8 +762,8 @@ glusterd_volume_start_glusterfs (glusterd_volinfo_t *volinfo, port = pmap_registry_alloc (THIS); GLUSTERD_GET_BRICK_PIDFILE (pidfile, path, brickinfo->hostname, count); - snprintf (volfile, PATH_MAX, "%s-%s-%d", brickinfo->hostname, - volinfo->volname, count); + snprintf (volfile, PATH_MAX, "%s.%s-%d", volinfo->volname, + brickinfo->hostname, count); snprintf (cmd_str, 8192, "%s/sbin/glusterfs --xlator-option server-*.listen-port=%d " -- cgit