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

Regression in docker volume CLI #691

Closed
kerneltime opened this issue Nov 3, 2016 · 1 comment
Closed

Regression in docker volume CLI #691

kerneltime opened this issue Nov 3, 2016 · 1 comment
Labels
kind/bug kind/usability

Comments

@kerneltime
Copy link
Contributor

kerneltime commented Nov 3, 2016 •
edited
Loading

The following command which used to work, no longer works due to how size suffix gets parsed.

root@photon-3Q686zl9Z [ ~ ]# docker volume create --driver=vmdk --name=MyVolume2 -o size=10gb
Error response from daemon: create MyVolume2: VolumeDriver.Create: Server returned an error: KeyError(u'gb',)

capatalization works

root@photon-3Q686zl9Z [ ~ ]# docker volume create --driver=vmdk --name=MyVolume2 -o size=10GB
MyVolume2
@lipingxue
Copy link
Contributor

LGTM

kerneltime pushed a commit that referenced this issue Nov 3, 2016
Merge pull request #692 from vmware/fix-docker-cli-regression.kerneltime
1c255bd 
Make size spec in Docker cli case independent. Fixes #691
@kerneltime kerneltime changed the title Regression is docker volume CLI Regression in docker volume CLI Nov 3, 2016
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Assignees
No one assigned
Labels
kind/bug kind/usability
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kerneltime @lipingxue