Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cleanup: correct cli --volumes-from description #7640

Merged
merged 1 commit into from Aug 20, 2014
Merged

Conversation

xiaods
Copy link
Contributor

@xiaods xiaods commented Aug 20, 2014

issue #7580 volumes-from comma separated list mentioned this case.

Options with [] may be specified multiple times.
Example:
case 1, works:
docker run -it --rm --volumes-from TEST_DATA --volumes-from TEST_DATA2 ubuntu
bash

case 2, failed:
docker run -it --rm --volumes-from TEST_DATA,TEST_DATA2 ubuntu bash
2014/08/20 15:40:35 Error response from daemon: Cannot start container
e2f63ce5611c7a1be28389580d4a3847713078e6d5b61aac6cd532ec68b02191: Container
TEST_DATA,TEST_DATA2 not found. Impossible to mount its volumes

Signed-off-by: Deshi Xiao dxiao@redhat.com

issue moby#7580 volumes-from comma separated list mentioned this case.

Options like --volumes-from=[] indicate they can be specified multiple times:
docker run -it --rm --volumes-from TEST_DATA --volumes-from TEST_DATA2 ubuntu bash

Signed-off-by: Deshi Xiao <dxiao@redhat.com>
@SvenDowideit
Copy link
Contributor

LGTM - @jamtur01 @fredlf @ostezer

@jamtur01
Copy link
Contributor

LGTM. Thanks. Be good if whitespace was separated into another commit in future.

jamtur01 added a commit that referenced this pull request Aug 20, 2014
Cleanup: correct cli --volumes-from description
@jamtur01 jamtur01 merged commit 9faf025 into moby:master Aug 20, 2014
@xiaods xiaods deleted the develop branch August 21, 2014 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants