diff options
| author | Amar Tumballi <amar@gluster.com> | 2010-08-26 05:53:41 +0000 | 
|---|---|---|
| committer | Anand V. Avati <avati@dev.gluster.com> | 2010-08-26 07:31:13 -0700 | 
| commit | 9fe52c92ace1c8c1e6e016aad8bb14973d6c9dc0 (patch) | |
| tree | 4934eac56bfe5aa6e4f098093b98fdd30d13b285 | |
| parent | 44f187fad77512e6b9d22852b79dc9621d688798 (diff) | |
glusterd: handle failure of 'getspec' in case of '-' in volume name
Signed-off-by: Amar Tumballi <amar@gluster.com>
Signed-off-by: Anand V. Avati <avati@dev.gluster.com>
BUG: 1433 ()
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=1433
| -rw-r--r-- | extras/volgen/CreateVolfile.py | 2 | ||||
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-handshake.c | 5 | ||||
| -rw-r--r-- | 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 "  | 
