docker: Use a list instead of a dict for volumes according to the docker-py docs. Do not add host-binds to volumes list.

This commit is contained in:
Stefan Junker 2015-03-12 12:55:14 +01:00
parent d8df5da49f
commit c3f92cca21

View file

@ -474,13 +474,13 @@ class DockerManager(object):
self.volumes = None self.volumes = None
if self.module.params.get('volumes'): if self.module.params.get('volumes'):
self.binds = {} self.binds = {}
self.volumes = {} self.volumes = []
vols = self.module.params.get('volumes') vols = self.module.params.get('volumes')
for vol in vols: for vol in vols:
parts = vol.split(":") parts = vol.split(":")
# regular volume # regular volume
if len(parts) == 1: if len(parts) == 1:
self.volumes[parts[0]] = {} self.volumes.append(parts[0])
# host mount (e.g. /mnt:/tmp, bind mounts host's /tmp to /mnt in the container) # host mount (e.g. /mnt:/tmp, bind mounts host's /tmp to /mnt in the container)
elif 2 <= len(parts) <= 3: elif 2 <= len(parts) <= 3:
# default to read-write # default to read-write