Skip to content
This repository has been archived by the owner on Nov 9, 2020. It is now read-only.

Datastore names with embedded spaces in them can't be used when creating a volume. #916

Closed
govint opened this issue Feb 16, 2017 · 2 comments

Comments

@govint
Copy link
Contributor

govint commented Feb 16, 2017

For example, docker volume create -driver vmdk -name vol@data\ store - won't work as the code doesn't handle datastore names with spaces in it. This can be fixed to allow such names. Or we can take the approach that the plugin only allows names without spaces in them - have a uniform naming convention for all names handled via the plugin - tenant, datastore names at least.

@govint govint added this to the 0.13 milestone Feb 16, 2017
@tusharnt tusharnt added the P0 label Feb 16, 2017
@tusharnt tusharnt removed the P1 label Feb 16, 2017
@tusharnt tusharnt modified the milestones: 0.12, 0.13 Feb 16, 2017
@lipingxue
Copy link
Contributor

Run the following test manually

Configuration:
two datastores "datastore1" and "datastore2" are accessible by VM "photon6"
VM "photon6" belongs to _DEFAULT tenant, and is created on "datastore1"

Step1: run "docker volume ls" to list current volume

root@photon-KwqUODFXp [ ~ ]# docker volume ls
DRIVER              VOLUME NAME
vsphere             vol1@datastore1
vsphere             vol2@datastore1

Step2: rename "datastore2" to "datastore 2", then create a volume with vol_name@datastore\ \ \ 2 (spaces are included in datastore name), volume creation succeeded.

root@photon-KwqUODFXp [ ~ ]# docker volume create --driver=vsphere --name vol3@datastore\ \ \ 2
vol3@datastore   2
root@photon-KwqUODFXp [ ~ ]# docker volume ls
DRIVER              VOLUME NAME
vsphere             vol1@datastore1
vsphere             vol2@datastore1
vsphere             vol3@datastore   2

@lipingxue
Copy link
Contributor

Fixed by this PR #988

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

No branches or pull requests

3 participants