Bug 1246024

Summary:	gluster commands space in brick path fails
Product:	[Community] GlusterFS	Reporter:	Saravanakumar <sarumuga>
Component:	cli	Assignee:	bugs <bugs>
Status:	CLOSED DEFERRED	QA Contact:
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	mainline	CC:	amukherj, atumball, bugs
Target Milestone:	---	Keywords:	Triaged
Target Release:	---
Hardware:	All
OS:	All
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2019-05-07 21:10:36 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Saravanakumar 2015-07-23 10:32:40 UTC

Description of problem:
1.
gluster volume create with spaces in directory in brick path fails.

2. If you run the same command with quotes for brick, it misunderstands brick path and creates a directory with escape character


Version-Release number of selected component (if applicable):


How reproducible:
# gluster volume create tv2 gfvm2:/opt/volume_test/tv_1/b\ 1 force
volume create: tv2: failed: Brick: gfvm2:/opt/volume_test/tv_1/b not available. Brick may be containing or be contained by an existing brick

# gluster volume create tv2 "gfvm2:/opt/volume_test/tv_1/b\ 1" force 
volume create: tv2: success: please start the volume to access data
# gluster volume info tv2
Volume Name: tv2
Type: Distribute
Volume ID: 6e07a8f5-6564-4951-8398-3b65825385bc
Status: Created
Number of Bricks: 1
Transport-type: tcp
Bricks:
Brick1: gfvm2:/opt/volume_test/tv_1/b\
Options Reconfigured:
performance.readdir-ahead: on


Steps to Reproduce:
1.gluster volume create tv2 gfvm2:/opt/volume_test/tv_1/b\ 1 force
2.gluster volume create tv2 "gfvm2:/opt/volume_test/tv_1/b\ 1" force 
3.

Actual results:
Errors as given above

Expected results:
1. If space is NOT accepted in brick path, it should be displayed properly.
2. I think escape character should NOT be used for creating a Brick.(as mentioned in gluster info <volume>) before.

Additional info:

Comment 2 Amar Tumballi 2019-05-07 21:10:36 UTC

This remains as a bug, but fixing this currently is not a priority among other things. It would become a major work, as everywhere we take brick as argument, we need to handle space.

We will add this in documentation, and mark this bug as DEFERRED, so we can revisit this after couple of releases.