Re: [jclouds] JCLOUDS-607: ComputeService.createNodesInGroup throws NPE on FloatingIPA... (#425)

2014-07-08 Thread Ignasi Barrera
I'd like it in 1.7.x. This issue affects also the Jenkins plugin and backporting will help implementing the changes on the Jenkins side. Thanks for your work here @cdancy! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/425#issuecomment-48276983

[jclouds-site] add docker guide (#110)

2014-07-08 Thread Andrea Turli
You can merge this Pull Request by running: git pull https://github.com/andreaturli/jclouds-site feature/docker Or you can view, comment on it, or merge it online at: https://github.com/jclouds/jclouds-site/pull/110 -- Commit Summary -- * add docker guide -- File Changes -- M

Re: [jclouds-site] add docker guide (#110)

2014-07-08 Thread CloudBees pull request builder plugin
[jclouds-site-pull-requests #351](https://jclouds.ci.cloudbees.com/job/jclouds-site-pull-requests/351/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-site/pull/110#issuecomment-48280249

Re: [jclouds-site] add docker guide (#110)

2014-07-08 Thread jclouds-commentator
Go to http://5350652a3b3a0f832b33-2bb942a57461dcb381b0c2aeb0a12b23.r79.cf5.rackcdn.com/ to review your changes. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-site/pull/110#issuecomment-48280414

Re: [jclouds-site] add docker guide (#110)

2014-07-08 Thread Ignasi Barrera
+permalink: /guides/docker/ +--- + +jclouds-docker is a local cloud provider modelled on [docker](http://www.docker.io). Similar to other jclouds supported +providers, it supports the same portable abstractions offered by jclouds. + +![jclouds docker architecture](/img/jclouds-docker.png)

Re: [jclouds-site] add docker guide (#110)

2014-07-08 Thread Ignasi Barrera
@@ -0,0 +1,43 @@ +--- +layout: page +title: Docker: Getting Started Guide +permalink: /guides/docker/ +--- + +jclouds-docker is a local cloud provider modelled on [docker](http://www.docker.io). Similar to other jclouds supported +providers, it supports the same portable abstractions

Re: [jclouds-site] add docker guide (#110)

2014-07-08 Thread Ignasi Barrera
+* Ensure you are using a recent JDK 6 +* Setup your project to include `docker` + * Get the dependency `org.jclouds.labs/docker` using jclouds [Installation](/start/install). +* Start coding + +{% highlight java %} +// get a context with docker that offers the portable ComputeService

Re: [jclouds-site] add docker guide (#110)

2014-07-08 Thread Ignasi Barrera
+ .modules(ImmutableSet.Module of(new Log4JLoggingModule(), +new SshjSshClientModule())) + .buildView(ComputeServiceContext.class); +ComputeService client = context.getComputeService(); +

Build failed in Jenkins: jclouds » jclouds CloudServers UK provider #2670

2014-07-08 Thread jenkins-no-reply
See https://jclouds.ci.cloudbees.com/job/jclouds/org.apache.jclouds.provider$cloudservers-uk/2670/ -- [INFO] [INFO]

Re: [jclouds-labs-google] JCLOUDS-458: Addressing suggestions in PR-31 (#33)

2014-07-08 Thread BuildHive
[jclouds » jclouds-labs-google #1107](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-labs-google/1107/) SUCCESS This pull request looks good [(what's this?)](https://www.cloudbees.com/what-is-buildhive) --- Reply to this email directly or view it on GitHub:

Re: [jclouds-labs-google] JCLOUDS-458: Addressing suggestions in PR-31 (#33)

2014-07-08 Thread CloudBees pull request builder plugin
[jclouds-labs-google-pull-requests #92](https://jclouds.ci.cloudbees.com/job/jclouds-labs-google-pull-requests/92/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-google/pull/33#issuecomment-48292319

Re: [jclouds-labs-google] JCLOUDS-458: Addressing suggestions in PR-31 (#33)

2014-07-08 Thread Bhathiya
+ BucketTemplateInternal template = new BucketTemplateInternal(src); + JsonObject bucketTemplate = (JsonObject) context.serialize(template, BucketTemplateInternal.class); + + // deal with bucketAccessControls + if (!(src.getAcl() == null)

Re: [jclouds-labs-google] JCLOUDS-458: Addressing suggestions in PR-31 (#33)

2014-07-08 Thread Bhathiya
+bucketTemplate.add(acl, null); + } + // deal with DefaultObjectAccessControls + if (!(src.getDefaultObjectAccessControls() == null) (src.getDefaultObjectAccessControls().isEmpty())) { +bucketTemplate.add(defaultObjectAccessControls, null);

Re: [jclouds] JCLOUDS-610: reboot, suspend, and resumeNodesMatching methods gain functionality similar to that of destroyNodesMatching (#419)

2014-07-08 Thread Christopher Dancy
Did this PR get merged? I ask only because the last github message here is noting there are unmerged commits. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/419#issuecomment-48326473

Re: [jclouds] JCLOUDS-610: reboot, suspend, and resumeNodesMatching methods gain functionality similar to that of destroyNodesMatching (#419)

2014-07-08 Thread Ignasi Barrera
Yes it is. You'll find it if you look at the history. When you rebase to the latest master, the commit hash changes, and that's why GitHub does not detect the PR as merged. Usually we ask contributors to rebase the branch before merging, but sometimes is just more agile to rebase to the latest

Re: [jclouds] JCLOUDS-610: reboot, suspend, and resumeNodesMatching methods gain functionality similar to that of destroyNodesMatching (#419)

2014-07-08 Thread Christopher Dancy
It does and thank you ;) --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/419#issuecomment-48329444

Re: [jclouds-labs-google] JCLOUDS-458: Addressing suggestions in PR-31 (#33)

2014-07-08 Thread Bhathiya
@@ -32,31 +30,30 @@ * @see a href= https://developers.google.com/storage/docs/cross-origin; / */ -public final class BucketCors { - private final SetString origins; - private final SetString methods; - private final SetString responseHeaders; +public class BucketCors { +

Re: [jclouds] JCLOUDS-607: ComputeService.createNodesInGroup throws NPE on FloatingIPA... (#425)

2014-07-08 Thread Andrew Phillips
I'd like it in 1.7.x. @nacx I tried to apply [the patch](https://issues.apache.org/jira/secure/attachment/12654411/JCLOUDS-607.patch) but it doesn't work on master. Could you submit a PR for 1.7.x..? --- Reply to this email directly or view it on GitHub:

Re: [jclouds] JCLOUDS-607: ComputeService.createNodesInGroup throws NPE on FloatingIPA... (#425)

2014-07-08 Thread Ignasi Barrera
NP! I'll do later today. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/425#issuecomment-48363551

Re: [jclouds] JCLOUDS-607: ComputeService.createNodesInGroup throws NPE on FloatingIPA... (#425)

2014-07-08 Thread Andrew Phillips
NP! I'll do later today. Thanks! ;-) --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/425#issuecomment-48367854

Re: [jclouds-labs-openstack] Neutron Refactoring (#111)

2014-07-08 Thread Zack Shoylev
@nacx Can you give this another look? Thanks! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-openstack/pull/111#issuecomment-48395860

Re: [jclouds-labs-openstack] Neutron Refactoring (#111)

2014-07-08 Thread Ignasi Barrera
+ +import com.google.common.base.Objects; + +/** + * A Neutron Subnet Allocation Pool + * + * @see a + * href=http://docs.openstack.org/api/openstack-network/2.0/content/Subnets.html;api + * doc/a + */ +public class AllocationPool { + + protected final String start; +

Re: [jclouds-labs-openstack] Neutron Refactoring (#111)

2014-07-08 Thread Ignasi Barrera
+ @SerializedName(n1kv:segment_add) + protected final String segmentAdd; + @SerializedName(n1kv:segment_del) + protected final String segmentDel; + @SerializedName(n1kv:member_segments) + protected final String memberSegments; + + // multiprovidernet.py: Multiprovider net

Re: [jclouds-labs-openstack] Neutron Refactoring (#111)

2014-07-08 Thread Ignasi Barrera
+ @GET + @Path(/{id}) + @ResponseParser(ParseRouters.class) + @Fallback(EmptyRoutersFallback.class) + Routers get(@PathParam(id) String id); + + /** +* Create a new router +* +* @param router Options for creating a router +* @return the newly created router

Re: [jclouds-labs-openstack] Neutron Refactoring (#111)

2014-07-08 Thread Ignasi Barrera
I had this PR in the bottom of the TODO list, sorry for my late reply! I like the new pattern you propose. The model and the APIs look cleaner. The code look good to me, just a couple comments: * There are several POST requests that have the ReturnNullOn404 fallback. Does the service really

Re: [jclouds-labs-openstack] Neutron Refactoring (#111)

2014-07-08 Thread Zack Shoylev
Thanks so much for having a look! ReturnNullOn404 - this needs to be fixed (I am pretty sure service does not return it, but still double checking some sources) @ConstructorProperties - based on previous discussions, I will add it back to domain classes. The plan is to rework those to follow

Re: [jclouds-labs-openstack] Neutron Refactoring (#111)

2014-07-08 Thread Zack Shoylev
+ +import com.google.common.base.Objects; + +/** + * A Neutron Subnet Allocation Pool + * + * @see a + * href=http://docs.openstack.org/api/openstack-network/2.0/content/Subnets.html;api + * doc/a + */ +public class AllocationPool { + + protected final String start; +

[jclouds] Remove ConsumeOnClose and add test (#435)

2014-07-08 Thread Andrew Gaul
You can merge this Pull Request by running: git pull https://github.com/andrewgaul/jclouds consume-on-close Or you can view, comment on it, or merge it online at: https://github.com/jclouds/jclouds/pull/435 -- Commit Summary -- * JCLOUDS-417: Remove ConsumeOnCloseInputStream *

Re: [jclouds] Remove ConsumeOnCloseInputStream (#243)

2014-07-08 Thread Andrew Gaul
Closed #243. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/243#event-139482818

Re: [jclouds] Remove ConsumeOnCloseInputStream (#243)

2014-07-08 Thread Andrew Gaul
@tdmanv Another user asked about cancellation on the jclouds user list and I revived your commit and added at test at #435. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/243#issuecomment-48410699

Re: [jclouds] Remove ConsumeOnClose and add test (#435)

2014-07-08 Thread Andrew Gaul
@tdmanv Please review. `git log` shows the correct attribution; unsure why GitHub shows @jenkins-maginatics. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/435#issuecomment-48410781

[jira] [Updated] (JCLOUDS-417) Payload.getInput.close reads all bytes from its InputStream

2014-07-08 Thread Andrew Gaul (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-417?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andrew Gaul updated JCLOUDS-417: Assignee: Thomas Manville Payload.getInput.close reads all bytes from its InputStream

[jira] [Updated] (JCLOUDS-417) Payload.getInput.close reads all bytes from its InputStream

2014-07-08 Thread Andrew Gaul (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-417?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andrew Gaul updated JCLOUDS-417: Summary: Payload.getInput.close reads all bytes from its InputStream (was:

[jira] [Updated] (JCLOUDS-417) Payload.getInput.close reads all bytes from its InputStream

2014-07-08 Thread Andrew Gaul (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-417?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andrew Gaul updated JCLOUDS-417: Description: Users do not expect this behavior when aborting getBlob operations. Further, by