You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@jclouds.apache.org by Andrew Donald Kennedy <no...@github.com> on 2017/03/17 01:54:37 UTC

[jclouds/jclouds] Fix error creating list of volume bindings for Docker (#1077)

Currently the code only sets the last in the list of the passed in volume bindings, by continually overwriting the bindings with a single element list. This fixes that and sets all of them in a single list.
You can view, comment on, or merge this pull request online at:

  https://github.com/jclouds/jclouds/pull/1077

-- Commit Summary --

  * Fix error creating list of volume bindings for Docker

-- File Changes --

    M apis/docker/src/main/java/org/jclouds/docker/compute/strategy/DockerComputeServiceAdapter.java (4)

-- Patch Links --

https://github.com/jclouds/jclouds/pull/1077.patch
https://github.com/jclouds/jclouds/pull/1077.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1077

Re: [jclouds/jclouds] Fix error creating list of volume bindings for Docker (#1077)

Posted by Andrew Donald Kennedy <no...@github.com>.

Hi, @andreaturli can you have a look at this pls? Not sure if it needs a new test?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1077#issuecomment-287245301

Re: [jclouds/jclouds] Fix error creating list of volume bindings for Docker (#1077)

Posted by Ignasi Barrera <no...@github.com>.

Pushed to master as [c3c86de9](http://git-wip-us.apache.org/repos/asf/jclouds/commit/c3c86de9). Thanks @grkvlt!

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1077#issuecomment-287485939

Re: [jclouds/jclouds] Fix error creating list of volume bindings for Docker (#1077)

Posted by Ignasi Barrera <no...@github.com>.

Closed #1077.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1077#event-1005239412