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

Minor fix for #817 - missing parentheses #924

Merged
merged 1 commit into from Feb 17, 2017
Merged

Conversation

pdhamdhere
Copy link
Contributor

Fixes #817

Without this change:

root@Photon-1 [ ~ ]# docker volume create -d vmdk --name=v1@bad
Error response from daemon: create v1@bad: VolumeDriver.Create: Server returned an error: TypeError('can only join an iterable',)

After this change:

root@Photon-1 [ ~ ]# docker volume create -d vmdk --name=v1@bad
Error response from daemon: create v1@bad: VolumeDriver.Create: Invalid datastore 'bad'.
Known datastores: datastore1, datastore2.
Default datastore: datastore1

Fixes #817

Without this change:
```
root@Photon-1 [ ~ ]# docker volume create -d vmdk --name=v1@bad
Error response from daemon: create v1@bad: VolumeDriver.Create: Server returned an error: TypeError('can only join an iterable',)
```

After this change:
```
root@Photon-1 [ ~ ]# docker volume create -d vmdk --name=v1@bad
Error response from daemon: create v1@bad: VolumeDriver.Create: Invalid datastore 'bad'.
Known datastores: datastore1, datastore2.
Default datastore: datastore1
```
Copy link
Contributor

@shuklanirdesh82 shuklanirdesh82 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! It helps user to know about valid values for datastores to try out.

@govint
Copy link
Contributor

govint commented Feb 17, 2017

Lets add a test case, this is an obvious scenario - missing datastore.

@shuklanirdesh82
Copy link
Contributor

shuklanirdesh82 commented Feb 17, 2017

Lets add a test case, this is an obvious scenario - missing datastore.

@govint thanks for pointing out .. it is already planned please refer #856

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

Successfully merging this pull request may close these issues.

None yet

4 participants