Problem mounting volumes after revamp

[ns-sjorgedeaguiar]

Hi, I was testing the new version after #575 and mounting is not working with custom images.

The problem seems to be because the runner sets the volumes config option:

// volumes
let vols: HashMap<String, HashMap<(), ()>> = runnable_image
    .volumes()
    .map(|(orig, dest)| (format!("{orig}:{dest}"), HashMap::new()))
    .collect();
config.volumes = Some(vols);

However that option is meant for anonymous volume mounts, where the keys are the path inside the container and the values are an empty map.
The binds hostconfig option is the one accepting a list of mounts. Replacing the above by something like this works:

// volumes
let binds: Vec<String> = runnable_image
    .volumes()
    .map(|(orig, dest)| format!("{orig}:{dest}"))
    .collect();
if !binds.is_empty() {
    config.host_config = config.host_config.map(|mut host_config| {
        host_config.binds = Some(binds);
        host_config
    });
}

[DDtKey]

Thanks for reporting!

Yeah, this should be fixed.

Interestingly, no one has used an http client with volumes before (that part hasn't actually been changed).
Anyway, the issue is pretty straightforward.

Feel free to open PR if you wish. Otherwise I'll address this a bit later