Bound volumes must be repeated

From the less verbose example of mounting volumes,

container_id = c.create_container(
    'busybox', 'ls', volumes=['/mnt/vol1', '/mnt/vol2'],
    host_config=docker.utils.create_host_config(binds=[
        '/home/user1/:/mnt/vol2',
        '/var/www:/mnt/vol1:ro',
    ])
)

we can see that '/mnt/vol1' and '/mnt/vol2' must be repeated. A preferred API would simply be

container_id = c.create_container(
    'busybox', 'ls',
    volumes=['/var/www:/mnt/vol1:ro', '/home/user1/:/mnt/vol2']
)

In this API, volumes

1. is actually a useful parameter when one needs to explicitly bind, without the verbosity of host_config and binds;
2. mirrors precisely the the API to docker run for mounting volumes, so that the user can re-apply the paradigm she has already learned; and
3. removes the need for redundantly specifying the same volume.

Additionally, volumes could support the more explicit dictionary argument, similar to binds:

container_id = c.create_container(
    'busybox', 'ls',
    volumes={
        '/home/user1/': {
            'bind': '/mnt/vol2',
            'mode': 'rw',
        },
        '/var/www': {
            'bind': '/mnt/vol1',
            'mode': 'ro',
        }
    }
)

[bfirsh]

Ah, sorry. It’s not actually released yet with documentation, but in 2.0 you’ll be able to do things along the lines of: client.containers.run("busybox", volumes=['/var/www:/mnt/vol1:ro', '/home/user1/:/mnt/vol2']). Ditto create().

[gotgenes]

@bfirsh Great! Thanks for following up!