You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@jclouds.apache.org by Zack Shoylev <no...@github.com> on 2016/03/18 21:54:57 UTC

[jclouds-labs] Update DockerComputeServiceAdapter.java (#247)

Is this needed for the compute service adapter?

It's causing issues against one of the docker service providers I am testing with.
You can view, comment on, or merge this pull request online at:

  https://github.com/jclouds/jclouds-labs/pull/247

-- Commit Summary --

  * Update DockerComputeServiceAdapter.java

-- File Changes --

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

-- Patch Links --

https://github.com/jclouds/jclouds-labs/pull/247.patch
https://github.com/jclouds/jclouds-labs/pull/247.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-labs/pull/247

Re: [jclouds-labs] Update DockerComputeServiceAdapter.java (#247)

Posted by Zack Shoylev <no...@github.com>.
@nacx I figured that out about the mirror. Fun github stuff :)

I like the idea of  doing this through the template options. I will look into that.

---
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/247#issuecomment-198544938

Re: [jclouds-labs] Update DockerComputeServiceAdapter.java (#247)

Posted by Ignasi Barrera <no...@github.com>.
@zack-shoylev push the branch to the ASF remote, otherwise the mirror job will delete it :)

Back to the PR, let's make that flag configurable from the Docker template options.

---
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/247#issuecomment-198543632