[Gluster-devel] Allow ":" as brick path

Niels de Vos ndevos at redhat.com
Sat Oct 29 11:58:43 UTC 2016

On Sat, Oct 29, 2016 at 01:07:33PM +0200, Gandalf Corvotempesta wrote:
> Would be possible to add support for ":" in brick path?
> i'm trying to use the following command:
> # gluster volume replace-brick gv0
> commit force
> wrong brick type:,
> use <HOSTNAME>:<export-dir-abs-path>
> Usage: volume replace-brick <VOLNAME> <SOURCE-BRICK> <NEW-BRICK> {commit force}
> but the new brick name is not reconized, probably because there are
> too many ":" in the argument.
> As the hostname (or the IP address) can't have ":", you should get the
> hostname/ip up to the first occurence of ":" and leaving the rest as
> brick name/path

Actually, IPv6 addresses would have ":" too. But, I'm not sure anyone
would want to write IPv6 addresses instead of hostnames.

I guess the parsing of the brick path could always use ":/" as separator
between the hostname and path. Please file a bug for this, as I also
agree that ":" should be allowed in the path.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://www.gluster.org/pipermail/gluster-devel/attachments/20161029/ce74f2ca/attachment-0001.sig>

More information about the Gluster-devel mailing list