[jira] [Updated] (JCLOUDS-767) Failed spot request but still created, node is null - unable to cancel

2014-11-19 Thread Yaron Rosenbaum (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-767?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Yaron Rosenbaum updated JCLOUDS-767: Labels: AWS DOS Spot (was: ) > Failed spot request but still created, node is null - unabl

[jira] [Commented] (JCLOUDS-767) Failed spot request but still created, node is null - unable to cancel

2014-11-19 Thread Yaron Rosenbaum (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-767?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14219079#comment-14219079 ] Yaron Rosenbaum commented on JCLOUDS-767: - Hi After further research with Amazon

[jira] [Commented] (JCLOUDS-775) aws-ec2 provider is rusting

2014-11-19 Thread Adrian Cole (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-775?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14219066#comment-14219066 ] Adrian Cole commented on JCLOUDS-775: - This particular error looks interesting. {cod

Re: [jclouds] JCLOUDS-480 start supporting version 4 signatures for aws-ec2. (#610)

2014-11-19 Thread Adrian Cole
Ran live tests, noting failures in JCLOUDS-775. No signature errors in logs, and out of almost 2500 requests, there was one 401, which resolved on the next request. I'm not saying this is bug-free code, just that I have confidence that once unit tests are in, this is safe to merge and backport.

[jira] [Commented] (JCLOUDS-775) aws-ec2 provider is rusting

2014-11-19 Thread Adrian Cole (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-775?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14219058#comment-14219058 ] Adrian Cole commented on JCLOUDS-775: - With code present in JCLOUDS-480 I have the fo

Jenkins build is back to stable : jclouds » jclouds #1944

2014-11-19 Thread BuildHive
See

Re: [jclouds] JCLOUDS-480 start supporting version 4 signatures for aws-ec2. (#610)

2014-11-19 Thread BuildHive
[jclouds » jclouds #1944](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1944/) 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: https://github.com/jclouds/jclouds/pull/610#issuecomme

Re: [jclouds] JCLOUDS-480 start supporting version 4 signatures for aws-ec2. (#610)

2014-11-19 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #1407](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1407/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/610#issuecomment-63764212

Re: [jclouds] JCLOUDS-480 start supporting version 4 signatures for aws-ec2. (#610)

2014-11-19 Thread CloudBees pull request builder plugin
[jclouds-pull-requests-java-7 #319](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests-java-7/319/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/610#issuecomment-63763880

Re: [jclouds] JCLOUDS-480 start supporting version 4 signatures for aws-ec2. (#610)

2014-11-19 Thread Adrian Cole
ok fixed the aws-ec2 tests so that they now pass with v4 signature. TODO: run live tests (which may be in very bad shape anyway) TODO: backfill unit test for FormSignerV4 --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/610#issuecomment-63763008

Jenkins build became unstable: jclouds » jclouds #1943

2014-11-19 Thread BuildHive
See

Re: [jclouds] JCLOUDS-480 start supporting version 4 signatures for aws-ec2. (#610)

2014-11-19 Thread BuildHive
[jclouds » jclouds #1943](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1943/) UNSTABLE Looks like there's a problem with this pull request [(what's this?)](https://www.cloudbees.com/what-is-buildhive) --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jcl

Re: [jclouds] JCLOUDS-781 cluster regions is a subset of regions. (#613)

2014-11-19 Thread BuildHive
[jclouds » jclouds #1942](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1942/) 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: https://github.com/jclouds/jclouds/pull/613#issuecomme

Jenkins build is back to stable : jclouds » jclouds #1942

2014-11-19 Thread BuildHive
See

Jenkins build is still unstable: jclouds » jclouds #1941

2014-11-19 Thread BuildHive
See

Re: [jclouds] JCLOUDS-480 start supporting version 4 signatures for aws-ec2. (#610)

2014-11-19 Thread BuildHive
[jclouds » jclouds #1941](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1941/) UNSTABLE Looks like there's a problem with this pull request [(what's this?)](https://www.cloudbees.com/what-is-buildhive) --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jcl

Jenkins build became unstable: jclouds » jclouds #1940

2014-11-19 Thread BuildHive
See

Re: [jclouds] JCLOUDS-480 start supporting version 4 signatures for aws-ec2. (#610)

2014-11-19 Thread BuildHive
[jclouds » jclouds #1940](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1940/) UNSTABLE Looks like there's a problem with this pull request [(what's this?)](https://www.cloudbees.com/what-is-buildhive) --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jcl

Re: [jclouds] JCLOUDS-480 start supporting version 4 signatures for aws-ec2. (#610)

2014-11-19 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #1406](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1406/) UNSTABLE Looks like there's a problem with this pull request --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/610#issuecomment-63750128

Re: [jclouds] JCLOUDS-480 start supporting version 4 signatures for aws-ec2. (#610)

2014-11-19 Thread CloudBees pull request builder plugin
[jclouds-pull-requests-java-7 #318](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests-java-7/318/) FAILURE Looks like there's a problem with this pull request --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/610#issuecomment-63749404

Re: [jclouds] JCLOUDS-781 cluster regions is a subset of regions. (#613)

2014-11-19 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #1405](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1405/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/613#issuecomment-63749267

Re: [jclouds-labs-openstack] Glance v2 implementation (#77)

2014-11-19 Thread Ignacio Mulas
Hi @inbarsto and @Everett ! I could not finish this as some other things came up by then and it has been hanging for a while now. I would actually like to finish it and push it into the repository. I can work on it during the next week so

Re: [jclouds] JCLOUDS-480 start supporting version 4 signatures for aws-ec2. (#610)

2014-11-19 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #1403](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1403/) UNSTABLE Looks like there's a problem with this pull request --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/610#issuecomment-63749043

Re: [jclouds] JCLOUDS-480 start supporting version 4 signatures for aws-ec2. (#610)

2014-11-19 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #1404](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1404/) UNSTABLE Looks like there's a problem with this pull request --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/610#issuecomment-63749045

Re: [jclouds] JCLOUDS-781 cluster regions is a subset of regions. (#613)

2014-11-19 Thread CloudBees pull request builder plugin
[jclouds-pull-requests-java-7 #317](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests-java-7/317/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/613#issuecomment-63748907

Build failed in Jenkins: jclouds-labs-google #333

2014-11-19 Thread jenkins-no-reply
See Changes: [Adrian Cole] Initial commit to add l7 loadbalancing features and tests to jclouds-labs-google [Adrian Cole] Added LICENSE file [Adrian Cole] l7 Cont: Removed ResourceView, Converted to AutoValue and updated.

Build failed in Jenkins: jclouds-labs-google » Apache jclouds labs google #333

2014-11-19 Thread jenkins-no-reply
See -- maven3-agent.jar already up to date maven3-interceptor.jar already up to date maven3-interceptor-commons.jar already up to date <===[JENKINS R

Re: [jclouds] JCLOUDS-480 start supporting version 4 signatures for aws-ec2. (#610)

2014-11-19 Thread CloudBees pull request builder plugin
[jclouds-pull-requests-java-7 #316](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests-java-7/316/) FAILURE Looks like there's a problem with this pull request --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/610#issuecomment-63748229

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread Adrian Cole
@ashmrtnz so.. a piece of you is now in jclouds! good work! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-google/pull/97#issuecomment-63747967

Re: [jclouds] JCLOUDS-480 start supporting version 4 signatures for aws-ec2. (#610)

2014-11-19 Thread CloudBees pull request builder plugin
[jclouds-pull-requests-java-7 #315](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests-java-7/315/) FAILURE Looks like there's a problem with this pull request --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/610#issuecomment-63748082

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread Adrian Cole
Thanks tons. this is truly a heroic save @danbroudy! Merged into master. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-google/pull/97#issuecomment-63747903

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread Adrian Cole
sounds same. will merge! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-google/pull/97#issuecomment-63747549

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread danbroudy
Ran the google-compute-engine test. ``` Results : Failed tests: GoogleComputeEngineServiceLiveTest>BaseComputeServiceLiveTest.testCreateAndRunAService:706->BaseComputeServiceLiveTest.createAndRunAServiceInGroup:723 » RunNodes GoogleComputeEngineServiceLiveTest>BaseComputeServiceLiveTest.

Re: [jclouds] JCLOUDS-480 start supporting version 4 signatures for aws-ec2. (#610)

2014-11-19 Thread Adrian Cole
no more expect tests in aws-ec2. next step, port the restannotationprocessor tests, --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/610#issuecomment-63746894

Re: [jclouds] Remove Rackspace Cloud Identity v1.1 (#614)

2014-11-19 Thread BuildHive
[jclouds » jclouds #1939](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1939/) 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: https://github.com/jclouds/jclouds/pull/614#issuecomme

Re: [jclouds] Remove Rackspace Cloud Identity v1.1 (#614)

2014-11-19 Thread Adrian Cole
+1 If you need help merging lemme know. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/614#issuecomment-63739427

Re: [jclouds] Remove Rackspace Cloud Identity v1.1 (#614)

2014-11-19 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #1402](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1402/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/614#issuecomment-63738876

Re: [jclouds] Remove Rackspace Cloud Identity v1.1 (#614)

2014-11-19 Thread CloudBees pull request builder plugin
[jclouds-pull-requests-java-7 #314](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests-java-7/314/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/614#issuecomment-63737932

Jenkins build is back to stable : jclouds » jclouds #1938

2014-11-19 Thread BuildHive
See

Re: [jclouds] JCLOUDS-781 cluster regions is a subset of regions. (#613)

2014-11-19 Thread BuildHive
[jclouds » jclouds #1938](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1938/) 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: https://github.com/jclouds/jclouds/pull/613#issuecomme

Re: [jclouds] Remove Rackspace Cloud Identity v1.1 (#614)

2014-11-19 Thread Everett Toews
Note that if there are any comments, I'll be back on Nov. 24 to address them. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/614#issuecomment-63736209

[jira] [Commented] (JCLOUDS-779) OpenStack/Rackspace providers might not honor jclouds.regions property

2014-11-19 Thread Everett Toews (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-779?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14218717#comment-14218717 ] Everett Toews commented on JCLOUDS-779: --- https://github.com/jclouds/jclouds/pull/61

[jclouds] Remove Rackspace Cloud Identity v1.1 (#614)

2014-11-19 Thread Everett Toews
You can merge this Pull Request by running: git pull https://github.com/rackspace/jclouds remove-rax-cloud-identity-1-1 Or you can view, comment on it, or merge it online at: https://github.com/jclouds/jclouds/pull/614 -- Commit Summary -- * Remove Rackspace Cloud Identity v1.1 -- File

Re: [jclouds-karaf] JCLOUDS-716: Remove Rackspace First-Gen Cloud Servers (#63)

2014-11-19 Thread Everett Toews
I'll leave it to someone with more knowledge of Karaf than I to actually merge this. Note that if there are any comments, I'll be back on Nov. 24 to address them. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-karaf/pull/63#issuecomment-63735574

Re: [jclouds-karaf] JCLOUDS-716: Remove Rackspace First-Gen Cloud Servers (#63)

2014-11-19 Thread BuildHive
[jclouds » jclouds-karaf #1452](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-karaf/1452/) 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: https://github.com/jclouds/jclouds-karaf/

Re: [jclouds] JCLOUDS-781 cluster regions is a subset of regions. (#613)

2014-11-19 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #1401](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1401/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/613#issuecomment-63732084

Re: [jclouds] JCLOUDS-781 cluster regions is a subset of regions. (#613)

2014-11-19 Thread CloudBees pull request builder plugin
[jclouds-pull-requests-java-7 #313](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests-java-7/313/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/613#issuecomment-63730673

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread BuildHive
[jclouds » jclouds-labs-google #1765](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-labs-google/1765/) 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: https://github.com/jclouds/jc

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread danbroudy
Addressed all comments and squashed. Now running live tests. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-google/pull/97#issuecomment-63729947

Re: [jclouds-labs-google] L7 rebased (#97)

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

Re: [jclouds] JCLOUDS-781 cluster regions is a subset of regions. (#613)

2014-11-19 Thread Adrian Cole
@demobox mind a quick look? my other pull request ran into this while working on mock tests :) --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/613#issuecomment-63729310

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread BuildHive
[jclouds » jclouds-labs-google #1764](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-labs-google/1764/) 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: https://github.com/jclouds/jc

Re: [jclouds-labs-google] L7 rebased (#97)

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

[jclouds] JCLOUDS-781 cluster regions is a subset of regions. (#613)

2014-11-19 Thread Adrian Cole
mild cleanup, but major change is to make sure the cluster region list is a subset of regions. Manually tested by invoking AWSEC2TemplateBuilderLiveTest with `-Djclouds.regions=us-east-1`. Verified in jclouds-wire.log that no calls were made to other regions. Mock test will happen in a future c

[jira] [Commented] (JCLOUDS-781) Impossibility to set jclouds.regions to a subset of EC2 regions while using EC2TemplateBuilder

2014-11-19 Thread Adrian Cole (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-781?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14218633#comment-14218633 ] Adrian Cole commented on JCLOUDS-781: - https://github.com/jclouds/jclouds/pull/613 >

Re: [jclouds-karaf] JCLOUDS-716: Remove Rackspace First-Gen Cloud Servers (#63)

2014-11-19 Thread CloudBees pull request builder plugin
[jclouds-karaf-pull-requests #105](https://jclouds.ci.cloudbees.com/job/jclouds-karaf-pull-requests/105/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-karaf/pull/63#issuecomment-63727162

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread Adrian Cole
> + @Produces(APPLICATION_JSON) > + @Path("/{backendService}") > + Operation patch(@PathParam("backendService") String backendServiceName, > + @BinderParam(BindToJsonPayload.class) > BackendServiceOptions backendServiceOptions); > + > + /** > +* Gets the most recent h

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread danbroudy
> + @Produces(APPLICATION_JSON) > + @Path("/{backendService}") > + Operation patch(@PathParam("backendService") String backendServiceName, > + @BinderParam(BindToJsonPayload.class) > BackendServiceOptions backendServiceOptions); > + > + /** > +* Gets the most recent h

[jclouds-karaf] JCLOUDS-716: Remove Rackspace First-Gen Cloud Servers (#63)

2014-11-19 Thread Everett Toews
You can merge this Pull Request by running: git pull https://github.com/everett-toews/jclouds-karaf JCLOUDS-716-remove-first-gen-cloud-servers Or you can view, comment on it, or merge it online at: https://github.com/jclouds/jclouds-karaf/pull/63 -- Commit Summary -- * JCLOUDS-716: Rem

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread BuildHive
[jclouds » jclouds-labs-google #1763](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-labs-google/1763/) 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: https://github.com/jclouds/jc

Re: [jclouds-labs-google] L7 rebased (#97)

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

[jira] [Commented] (JCLOUDS-781) Impossibility to set jclouds.regions to a subset of EC2 regions while using EC2TemplateBuilder

2014-11-19 Thread Adrian Cole (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-781?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14218573#comment-14218573 ] Adrian Cole commented on JCLOUDS-781: - So the cluster images property (jclouds.ec2.cc

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread Adrian Cole
> +/** > + * Represents a url map resource. > + * > + * @see href="https://developers.google.com/compute/docs/reference/latest/urlMaps"/> > + * @see href="https://developers.google.com/compute/docs/load-balancing/http/url-map"/> > + */ > +@AutoValue > +public abstract class UrlMap { > + > + pub

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread BuildHive
[jclouds » jclouds-labs-google #1762](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-labs-google/1762/) 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: https://github.com/jclouds/jc

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread danbroudy
> + /** > +* @see > org.jclouds.googlecomputeengine.domain.BackendService#getFingerprint() > +*/ > + public String getFingerprint() { > + return fingerprint; > + } > + > + /** > +* @see > org.jclouds.googlecomputeengine.domain.BackendService#getFingerprint() > +*/ > +

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread danbroudy
> + > +import static com.google.common.base.Objects.equal; > + > +import java.net.URI; > +import java.util.List; > + > +import org.jclouds.googlecomputeengine.domain.BackendService.Backend; > + > +import com.google.common.base.Objects; > + > +/** > + * Options to create a backend service resource.

Re: [jclouds-labs-google] L7 rebased (#97)

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

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread danbroudy
> +* > +* @param namethe name of the backend service to be inserted. > +* @param backendService options for this backend service. > +* @return an Operation resource. To check on the status of an operation, > +* poll the Operations resource returned to you, a

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread Adrian Cole
> + public String getDescription(){ > + return description; > + } > + > + /** > +* @see BackendServiceOptions#getDescription() > +*/ > + public BackendServiceOptions description(String description) { > + this.description = description; > + return this; > + } > + > +

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread danbroudy
> + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. > + * See the License for the specific language governing permissions and > + * limitations under the License. > + */ > +package org.jclouds.googlecomputeengine.domain; > + > +import java.net.URI; > +import java.util.Dat

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread danbroudy
> + @Nullable String fingerprint) { > + return new AutoValue_UrlMap(id, creationTimestamp, selfLink, name, > description, hostRules, pathMatchers, > +urlMapTests, defaultService, fingerprint); > + } > + > + UrlMap(){ > + } > + > + /** > +* An urlMap hostR

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread danbroudy
> + assertNull(api.delete("jclouds-test")); > + } > + > + public void testListBackendServiceResponseIs2xx() { > + HttpRequest request = getBasicRequest().method("GET") > + .endpoint(ENDPOINT_BASE) > + .build(); > + HttpResponse response = createRespons

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread danbroudy
> @@ -47,7 +48,10 @@ > > @Nullable public abstract String description(); > > - public abstract URI region(); > + public abstract Date creationTimestamp(); > + > + // This is @Nullable to support globalForwardingRules as well Done. --- Reply to this email directly or view it on GitHu

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread danbroudy
> + * distributed under the License is distributed on an "AS IS" BASIS, > + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. > + * See the License for the specific language governing permissions and > + * limitations under the License. > + */ > +package org.jclouds.googlec

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread danbroudy
> +/** > + * Represents a url map resource. > + * > + * @see href="https://developers.google.com/compute/docs/reference/latest/urlMaps"/> > + * @see href="https://developers.google.com/compute/docs/load-balancing/http/url-map"/> > + */ > +@AutoValue > +public abstract class UrlMap { > + > + pub

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread danbroudy
> + public abstract String name(); > + @Nullable public abstract String description(); > + public abstract URI defaultService(); > + public abstract List pathRules(); > + > + @SerializedNames({ "name", "description", "defaultService", > "pathRules" }) > + public stati

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread danbroudy
> + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. > + * See the License for the specific language governing permissions and > + * limitations under the License. > + */ > +package org.jclouds.googlecomputeengine.domain; > + > +import java.net.URI; > +import java.util.Lis

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread danbroudy
> +import org.jclouds.googlecomputeengine.options.ListOptions; > +import org.jclouds.javax.annotation.Nullable; > +import org.jclouds.oauth.v2.filters.OAuthFilter; > +import org.jclouds.rest.annotations.BinderParam; > +import org.jclouds.rest.annotations.Fallback; > +import org.jclouds.rest.annotat

Re: [jclouds] JCLOUDS-480 start supporting version 4 signatures for aws-ec2. (#610)

2014-11-19 Thread BuildHive
[jclouds » jclouds #1937](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1937/) UNSTABLE Looks like there's a problem with this pull request [(what's this?)](https://www.cloudbees.com/what-is-buildhive) --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jcl

Jenkins build is unstable: jclouds » jclouds #1937

2014-11-19 Thread BuildHive
See

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread BuildHive
[jclouds » jclouds-labs-google #1761](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-labs-google/1761/) 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: https://github.com/jclouds/jc

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread BuildHive
[jclouds » jclouds-labs-google #1760](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-labs-google/1760/) 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: https://github.com/jclouds/jc

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread danbroudy
> + public String getDescription(){ > + return description; > + } > + > + /** > +* @see BackendServiceOptions#getDescription() > +*/ > + public BackendServiceOptions description(String description) { > + this.description = description; > + return this; > + } > + > +

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread danbroudy
> + @Produces(APPLICATION_JSON) > + @Path("/{backendService}") > + Operation patch(@PathParam("backendService") String backendServiceName, > + @BinderParam(BindToJsonPayload.class) > BackendServiceOptions backendServiceOptions); > + > + /** > +* Gets the most recent h

Re: [jclouds-labs-google] L7 rebased (#97)

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

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread danbroudy
> + * > + * Unless required by applicable law or agreed to in writing, software > + * distributed under the License is distributed on an "AS IS" BASIS, > + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. > + * See the License for the specific language governing permission

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread danbroudy
> +import org.testng.annotations.Test; > + > +import com.google.common.collect.ImmutableList; > + > +@Test(groups = "unit") > +public class ParseUrlMapValidateTest extends > BaseGoogleComputeEngineParseTest { > + > + @Override > + public String resource() { > + return "/url_map_validate.j

Re: [jclouds-labs-google] L7 rebased (#97)

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

Re: [jclouds] JCLOUDS-480 start supporting version 4 signatures for aws-ec2. (#610)

2014-11-19 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #1400](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1400/) UNSTABLE Looks like there's a problem with this pull request --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/610#issuecomment-63709797

Re: [jclouds] JCLOUDS-480 start supporting version 4 signatures for aws-ec2. (#610)

2014-11-19 Thread CloudBees pull request builder plugin
[jclouds-pull-requests-java-7 #312](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests-java-7/312/) FAILURE Looks like there's a problem with this pull request --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/610#issuecomment-63708440

Re: [jclouds] JCLOUDS-480 start supporting version 4 signatures for aws-ec2. (#610)

2014-11-19 Thread Adrian Cole
added first mock compute service tests. There's a monster left AWSEC2ComputeServiceExpectTest, and then a bunch of RestAnnotaionProcessor tests, which I may have to just mildly adapt as I don't have 8 hours to rewrite them all and also be able to address the tons of other fixes we need. --- Rep

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread Adrian Cole
> + /** > +* @see > org.jclouds.googlecomputeengine.domain.BackendService#getFingerprint() > +*/ > + public String getFingerprint() { > + return fingerprint; > + } > + > + /** > +* @see > org.jclouds.googlecomputeengine.domain.BackendService#getFingerprint() > +*/ > +

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread Adrian Cole
> + /** > +* @see > org.jclouds.googlecomputeengine.domain.BackendService#getFingerprint() > +*/ > + public String getFingerprint() { > + return fingerprint; > + } > + > + /** > +* @see > org.jclouds.googlecomputeengine.domain.BackendService#getFingerprint() > +*/ > +

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread danbroudy
> + /** > +* @see > org.jclouds.googlecomputeengine.domain.BackendService#getFingerprint() > +*/ > + public String getFingerprint() { > + return fingerprint; > + } > + > + /** > +* @see > org.jclouds.googlecomputeengine.domain.BackendService#getFingerprint() > +*/ > +

Build failed in Jenkins: jclouds » jclouds #1936

2014-11-19 Thread BuildHive
See Changes: [Adrian Cole] JCLOUDS-480 start supporting version 4 signatures for aws-ec2. -- [...truncated 27613 lines...] [INFO] jclouds skeletons project . SKI

Re: [jclouds] JCLOUDS-480 start supporting version 4 signatures for aws-ec2. (#610)

2014-11-19 Thread BuildHive
[jclouds » jclouds #1936](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1936/) FAILURE Looks like there's a problem with this pull request [(what's this?)](https://www.cloudbees.com/what-is-buildhive) --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclo

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread danbroudy
Thanks for the speedy review! I'm on it. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-google/pull/97#issuecomment-63697668

Re: [jclouds] JCLOUDS-480 start supporting version 4 signatures for aws-ec2. (#610)

2014-11-19 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #1399](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1399/) FAILURE Looks like there's a problem with this pull request --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/610#issuecomment-63695368

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread Adrian Cole
OK all done review. Things I mentioned as not TODO I feel could be addressed before merge. We can do further cleanup after merge. nice save @danbroudy! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-google/pull/97#issuecomment-63695235

Re: [jclouds-labs-google] L7 rebased (#97)

2014-11-19 Thread Adrian Cole
> +import org.testng.annotations.Test; > + > +import com.google.common.collect.ImmutableList; > + > +@Test(groups = "unit") > +public class ParseUrlMapValidateTest extends > BaseGoogleComputeEngineParseTest { > + > + @Override > + public String resource() { > + return "/url_map_validate.j

  1   2   >