Skip to content

Add option to configure devices in yaml file#469

Closed
twendt wants to merge 1 commit into
docker:masterfrom
twendt:master
Closed

Add option to configure devices in yaml file#469
twendt wants to merge 1 commit into
docker:masterfrom
twendt:master

Conversation

@twendt
Copy link
Copy Markdown

@twendt twendt commented Sep 4, 2014

Example:
tvheadend:
image: tvheadend
devices:

  • "/dev/dvb/adapter0/dvr0:/dev/dvb/adapter0/dvr0"

Format for each entry is HOST:CONTAINER.
If only HOST is given then the file will be the same
inside the container.

Example:
tvheadend:
  image: tvheadend
  devices:
  - "/dev/dvb/adapter0/dvr0:/dev/dvb/adapter0/dvr0"

Format for each entry is HOST:CONTAINER.
If only HOST is given then the file will be the same
inside the container.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Note, this is actually a vendored version of https://github.com/docker/docker-py

These changes should be made upstream

@twendt
Copy link
Copy Markdown
Author

twendt commented Sep 4, 2014

Thank you for the info. I didn't notice that. Is there a certain reason why there is a copy of it in here?
I would try to bring it upstream, but I don't really know how to get started with it. I will see what I can do.

@dnephin
Copy link
Copy Markdown

dnephin commented Sep 4, 2014

Check out the discussion in #375

You might be able to just generate a patch with git diff and try to git apply it.

@twendt
Copy link
Copy Markdown
Author

twendt commented Sep 5, 2014

I requested the changes upstream now. How do we get the new version of docker-py in here then once the change hopefully got merged?

@ghost
Copy link
Copy Markdown

ghost commented Dec 22, 2014

+1 .. need to bind to /dev/vboxdrv

@bfirsh
Copy link
Copy Markdown

bfirsh commented Mar 6, 2015

Hi @twendt – new version of docker-py should be in Compose now! Would love to get this in.

@vdemeester
Copy link
Copy Markdown

As #1400 has been merged, I guess this PR could/should be closed ;).

@aanand aanand closed this May 12, 2015
infraAnchor pushed a commit to infraAnchor/compose that referenced this pull request Mar 6, 2026
Signed-off-by: Simon Pasquier <spasquie@redhat.com>
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.

5 participants