> @@ -136,6 +140,14 @@ public DockerComputeServiceAdapter(DockerApi api) {
>                .publishAllPorts(true)
>                .privileged(true);
>  
> +      if (templateOptions.getDirectPorts().isPresent()) {

Agree. Could have a `Map<Integer, Integer>` type here, instead? This would be 
mappings of host port to container port (this is the order  Docker uses with 
`--publish`.)

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/162/files#r27834588

Reply via email to