Skip to content

Add --device flag to allow additional host devices in container#6961

Merged
vieux merged 1 commit intomoby:masterfrom
crosbymichael:devices-flag
Jul 11, 2014
Merged

Add --device flag to allow additional host devices in container#6961
vieux merged 1 commit intomoby:masterfrom
crosbymichael:devices-flag

Conversation

@crosbymichael
Copy link
Copy Markdown
Contributor

We add a --device flag which can be used like:

docker run --device /dev/sda:/dev/xvda:rwm ubuntu /bin/bash

To allow the container to have read write permissions to access the host's /dev/sda via a node named /dev/xvda in the container.

Note: Much of this code was written by Dinesh Subhraveti [email protected] (github: dineshs-altiscale) and so he deserves a ton of credit.

This is a rebase of #6134

Closes #6134

Docker-DCO-1.1-Signed-off-by: Timothy [email protected] (github: timthelion)

We add a --device flag which can be used like:

 docker run --device /dev/sda:/dev/xvda:rwm ubuntu /bin/bash

To allow the container to have read write permissions to access the host's /dev/sda via a node named /dev/xvda in the container.

Note: Much of this code was written by Dinesh Subhraveti [email protected] (github: dineshs-altiscale) and so he deserves a ton of credit.

Docker-DCO-1.1-Signed-off-by: Timothy <[email protected]> (github: timthelion)
@crosbymichael
Copy link
Copy Markdown
Contributor Author

LGTM

1 similar comment
@vieux
Copy link
Copy Markdown
Contributor

vieux commented Jul 11, 2014

LGTM

vieux added a commit that referenced this pull request Jul 11, 2014
Add --device flag to allow additional host devices in container
@vieux vieux merged commit 9216ced into moby:master Jul 11, 2014
@vieux
Copy link
Copy Markdown
Contributor

vieux commented Jul 11, 2014

@fredlf @jamtur01 sorry for merging this, @crosbymichael tricked me. I thought you already validated in #6134 like @SvenDowideit

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.

3 participants