Re: [jclouds] Update checkstyle plugin dependency config (#288)

Didn't know that config property existed. I'd add it here too. Thanks for checking! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/288#issuecomment-34846172

Re: [jclouds] Update checkstyle plugin dependency config (#288)

Thanks for checking, by the way! On an unrelated note, I see that the core Checkstyle config now [also has](https://github.com/jclouds/jclouds/blob/master/project/pom.xml#L662): ``` true ``` Should we add this here? --- Reply to this email directly or view it on GitHub: https://github.com/jcloud

Re: [jclouds] Update checkstyle plugin dependency config (#288)

> The jclouds-chef/project module also has the header file but that shouldn't > be a problem. Ah, that's a good point. But it seems like we should be OK: Result of `mvn help:active-profiles` in jclouds-chef/project ``` [INFO] Active Profiles for Project 'org.apache.jclouds.chef:chef-project:pom

Re: [jclouds] Update checkstyle plugin dependency config (#288)

I'm a bit late but it looks good :) The jclouds-chef/project module also has the header file but that shouldn't be a problem. As long as projects in other repos get the checkstyle config from the classpath, the change is ok! --- Reply to this email directly or view it on GitHub: https://github.

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Committed to 1.7.x: [#1](https://git-wip-us.apache.org/repos/asf?p=jclouds.git;a=commit;h=4804edb7e873b9ce7161a38a69cf1706106c1d89), [#2](https://git-wip-us.apache.org/repos/asf?p=jclouds.git;a=commit;h=b1a2baee6227cd9cbad87189fb6b54747cbc8cdd) --- Reply to this email directly or view it on Git

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

[jclouds » jclouds #838](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/838/) 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/284#issuecomment

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

[jclouds-java-7-pull-requests #1068](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/1068/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/284#issuecomment-34844510

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

[jclouds-pull-requests #597](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/597/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/284#issuecomment-34844265

Re: [jclouds] Improve use of Splitter in Atmos code (53134df)

+1 - thanks, @andrewgaul! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/commit/53134dfa4efa499d21a9b9376c071a37ee047d35#commitcomment-5348973

Jenkins build is back to stable : jclouds #2539

See

Jenkins build is back to stable : jclouds » jclouds openstack-nova api #2539

See

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

See

[jira] [Commented] (JCLOUDS-137) Atmos should retry on HTTP 500 AtmosError 1040

[ https://issues.apache.org/jira/browse/JCLOUDS-137?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13898829#comment-13898829 ] ASF subversion and git services commented on JCLOUDS-137: - Commit

Re: [jclouds] Update checkstyle plugin dep config (1.7.x) (#289)

Committed to 1.7.x: [#1](https://git-wip-us.apache.org/repos/asf?p=jclouds.git;a=commit;h=5f7a4aa5bf68cf1011ed7142265effc37c95908b), [#2](https://git-wip-us.apache.org/repos/asf?p=jclouds.git;a=commit;h=4804edb7e873b9ce7161a38a69cf1706106c1d89) --- Reply to this email directly or view it on Git

[jira] [Commented] (JCLOUDS-450) Support min/max count and client token fields for AWS EC2 RunInstances

[ https://issues.apache.org/jira/browse/JCLOUDS-450?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13898830#comment-13898830 ] ASF subversion and git services commented on JCLOUDS-450: - Commit

Jenkins build is back to stable : jclouds-1.6.x-branch » jclouds Components Core #256

See

Jenkins build is back to stable : jclouds-1.6.x-branch #256

See

Re: [jclouds] Update checkstyle plugin dep config (1.7.x) (#289)

> jclouds » jclouds #836 UNSTABLE Unrelated [test failure](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/org.apache.jclouds$jclouds-compute/836/testReport/junit/org.jclouds.compute.callables/BlockUntilInitScriptStatusIsZeroThenReturnOutputTest/testloopUntilTrueOrThrowCancellationExcepti

Re: [jclouds] Update checkstyle plugin dep config (1.7.x) (#289)

[jclouds » jclouds #836](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/836/) 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/jclou

Jenkins build became unstable: jclouds » jclouds #836

See

Jenkins build is back to normal : jclouds-cli-1.7.x-branch #97

See

Jenkins build is back to normal : jclouds-karaf-1.7.x-branch #113

See

Re: [jclouds] Update checkstyle plugin dependency config (#288)

Committed to master: [#1](https://git-wip-us.apache.org/repos/asf?p=jclouds.git;a=commit;h=d7d76291ae244ee2bd03b59cc1a5157b599f7d58), [#2](https://git-wip-us.apache.org/repos/asf?p=jclouds.git;a=commit;h=a1fc6558edb7d6c5fa6d490243e3ae8321864ad5). @nacx Please let me know if you have any questio

Re: [jclouds] Update checkstyle plugin dep config (1.7.x) (#289)

[jclouds-pull-requests #594](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/594/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/289#issuecomment-34839533

Re: [jclouds] Update checkstyle plugin dependency config (#288)

[jclouds-java-7-pull-requests #1067](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/1067/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/288#issuecomment-34839365

Re: [jclouds] Update checkstyle plugin dependency config (#288)

[jclouds-pull-requests #595](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/595/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/288#issuecomment-34839287

Re: [jclouds] Update checkstyle plugin dep config (1.7.x) (#289)

[jclouds-java-7-pull-requests #1066](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/1066/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/289#issuecomment-34839053

Re: [jclouds] Update checkstyle plugin dependency config (#288)

[jclouds » jclouds #835](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/835/) 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/288#issuecomment

Jenkins build is back to normal : jclouds-labs-aws-1.7.x-branch #22

See

Jenkins build is back to normal : jclouds-labs-aws-1.7.x-branch » Apache jclouds labs aws #22

See

Jenkins build is back to normal : jclouds-labs-aws-1.7.x-branch #22

See

Jenkins build is back to normal : jclouds-labs-google-1.7.x-branch #66

See

Jenkins build is back to normal : jclouds-labs-google-1.7.x-branch » Apache jclouds labs google #66

See

Jenkins build is back to normal : jclouds-labs-1.7.x-branch #26

See

Jenkins build is back to normal : jclouds-labs-1.7.x-branch » Apache jclouds labs #26

See

Build failed in Jenkins: jclouds-labs-1.7.x-branch #25

See -- [...truncated 679 lines...] mvn install:install-file -DgroupId=org.apache.jclouds.labs -DartifactId=cloudsigma2 -Dversion=1.7.2-SNAPSHOT -Dpackaging=jar -Dfile=/path/to/file

Build failed in Jenkins: jclouds-labs-1.7.x-branch » Apache jclouds labs #25

See -- [...truncated 321 lines...] Downloading: https://oss.sonatype.org/content/repositories/snapshots/org/apache/jclouds/labs/fgcp/1.7.2-SNAPSHOT/ma

Re: [jclouds] Update checkstyle plugin dependency config (#288)

> Unrelated [spurious test failure](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/org.apache.jclouds$jclouds-compute/592/testReport/org.jclouds.compute.callables/BlockUntilInitScriptStatusIsZeroThenReturnOutputTest/testloopUntilTrueOrThrowCancellationExceptionReturnsWhenPredicateIsT

Re: [jclouds] Update checkstyle plugin dependency config (#288)

@nacx Care to have a quick look, seeing as you committed 335f59? Thanks! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/288#issuecomment-34836505

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

[jclouds » jclouds #834](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/834/) 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/284#issuecomment

Re: [jclouds] Update checkstyle plugin dependency config (#288)

[jclouds-pull-requests #592](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/592/) 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/288#issuecomment-34835263

Re: [jclouds] Update checkstyle plugin dependency config (#288)

[jclouds-java-7-pull-requests #1064](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/1064/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/288#issuecomment-34834364

Re: [jclouds] Update checkstyle plugin dep config (1.7.x) (#289)

[jclouds-java-7-pull-requests #1065](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/1065/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/289#issuecomment-34834363

Re: [jclouds] Update checkstyle plugin dep config (1.7.x) (#289)

[jclouds-pull-requests #593](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/593/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/289#issuecomment-34834252

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

[jclouds-pull-requests #591](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/591/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/284#issuecomment-34833767

Re: [jclouds] Update checkstyle plugin dep config (1.7.x) (#289)

[jclouds-pull-requests #590](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/590/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/289#issuecomment-34833378

Re: [jclouds] Update checkstyle plugin dependency config (#288)

[jclouds-java-7-pull-requests #1061](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/1061/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/288#issuecomment-34833449

Re: [jclouds] Update checkstyle plugin dep config (1.7.x) (#289)

[jclouds-java-7-pull-requests #1062](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/1062/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/289#issuecomment-34833448

Re: [jclouds] Update checkstyle plugin dependency config (#288)

[jclouds-pull-requests #589](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/589/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/288#issuecomment-34833377

Re: [jclouds] Update checkstyle plugin dependency config (#288)

[jclouds » jclouds #833](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/833/) 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/288#issuecomment

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

[jclouds-java-7-pull-requests #1063](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/1063/) 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/284#issuecomment-34831663

Re: [jclouds] Update checkstyle plugin dependency config (#288)

Not sure we _need_ the plugin repo change for this, actually...let me see if we can remove it entirely. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/288#issuecomment-34831513

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

> jclouds-pull-requests #588 SUCCESS Weird. Maybe we don't need the repo thing after all. Let's try one more time... --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/284#issuecomment-34831412

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Closed #284. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/284

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Reopened #284. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/284

Jenkins build is back to normal : jclouds-labs-openstack-1.7.x-branch #32

See

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

[jclouds-pull-requests #588](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/588/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/284#issuecomment-34831141

Re: [jclouds-chef] Adding 2009 to the project POM (#40)

Committed to [1.7.x](https://git-wip-us.apache.org/repos/asf?p=jclouds-chef.git;a=commit;h=fd97267b9f0b14c77acf406ff45ae46074d1) --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-chef/pull/40#issuecomment-34831044

Jenkins build is back to normal : jclouds-chef-1.7.x-branch #81

See

Jenkins build is back to normal : jclouds-chef-1.7.x-branch » Apache jclouds Chef :: Core #81

See

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

[jclouds » jclouds #832](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/832/) 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/284#issuecomment

[jclouds] Update checkstyle plugin dependency config (#288)

The problem is that, even though the Checkstyle plugin config in the jclouds-project profile is correctly applied, the effective POM in jclouds-project _still_ contains the dependency, even though it doesn't need it: maven-checkstyle-plugin

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

> Looks like it's looking in the wrong repo: Waiting on https://github.com/jclouds/jclouds/pull/289 --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/284#issuecomment-34830859

[jclouds] Update checkstyle plugin dep config (1.7.x) (#289)

Backport of https://github.com/jclouds/jclouds/pull/288 to 1.7.x You can merge this Pull Request by running: git pull https://github.com/jclouds/jclouds update-checkstyle-config-1.7.x Or you can view, comment on it, or merge it online at: https://github.com/jclouds/jclouds/pull/289 -- Commi

Build failed in Jenkins: jclouds-labs-aws-1.7.x-branch #21

See -- [...truncated 2193 lines...] [ERROR] from the specified remote repositories: [ERROR] central (http://repo.cloudbees.com/content/repositories/central, releases=true, snapshots=fa

Re: [jclouds-chef] Adding 2009 to the project POM (#40)

[jclouds-chef-pull-requests #103](https://jclouds.ci.cloudbees.com/job/jclouds-chef-pull-requests/103/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-chef/pull/40#issuecomment-34830801

Build failed in Jenkins: jclouds-labs-aws-1.7.x-branch » Apache jclouds labs aws #21

See -- [...truncated 1860 lines...] Downloading: https://repository-jclouds.forge.cloudbees.com/snapshot/org/apache/jclouds/labs/rds/1.7.2-SNA

Build failed in Jenkins: jclouds-labs-aws-1.7.x-branch #21

See -- [...truncated 2192 lines...] [ERROR] [ERROR] from the specified remote repositories: [ERROR] central (http://repo.cloudbees.com/content/repositories/central, releases=true, sna

Jenkins build is back to normal : jclouds » jclouds-chef #769

See

Re: [jclouds-chef] Adding 2009 to the project POM (#40)

[jclouds » jclouds-chef #769](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-chef/769/) 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-chef/pull/4

Build failed in Jenkins: jclouds-labs-google-1.7.x-branch » Apache jclouds labs google #65

See -- maven3-agent.jar already up to date maven3-interceptor.jar already up to date maven3-interceptor-commons.jar already up to date <=

Build failed in Jenkins: jclouds-labs-google-1.7.x-branch #65

See -- [...truncated 240 lines...] [ERROR] [ERROR] Path to dependency: [ERROR] 1) org.apache.jclouds.labs:google-compute-engine:jar:1.7.2-SNAPSHOT [ERROR] 2) org.apache.jclouds.labs

Jenkins build is back to normal : jclouds-1.7.x-branch » Apache jclouds Project #43

See

Jenkins build is back to normal : jclouds-1.7.x-branch #43

See

Jenkins build is back to normal : jclouds-1.7.x-branch » jclouds Components Core #43

See

Re: [jclouds-chef] Adding 2009 to the project POM (#40)

Reopened #40. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-chef/pull/40

Re: [jclouds-chef] Adding 2009 to the project POM (#40)

Closed #40. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-chef/pull/40

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

> jclouds-java-7-pull-requests #1060 FAILURE Still a problem with Checkstyle deps. Looks like it's looking in the wrong repo: ``` Caused by: org.apache.maven.plugin.PluginResolutionException: Plugin org.apache.maven.plugins:maven-checkstyle-plugin:2.11 or one of its dependencies could not be res

Re: [jclouds-chef] Adding 2009 to the project POM (#40)

> jclouds-chef-pull-requests #102 FAILURE Eek. Jumped the gun a bit there ;-) Still missing some deps: ``` Caused by: org.sonatype.aether.transfer.ArtifactNotFoundException: Could not find artifact org.apache.jclouds:jclouds-allcompute:jar:1.7.2-SNAPSHOT ``` --- Reply to this email directly or v

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

[jclouds-java-7-pull-requests #1060](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/1060/) 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/284#issuecomment-34828786

Re: [jclouds-chef] Adding 2009 to the project POM (#40)

[jclouds-chef-pull-requests #102](https://jclouds.ci.cloudbees.com/job/jclouds-chef-pull-requests/102/) 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-chef/pull/40#issuecomment-34828607

Build failed in Jenkins: jclouds » jclouds-chef #768

See -- [...truncated 581 lines...] Starting test testOneRecipeOmnibus(org.jclouds.chef.functions.GroupToBootScriptTest) Starting test testMustHaveRunScriptsName(org.jclouds.chef.functions.

Re: [jclouds-chef] Adding 2009 to the project POM (#40)

[jclouds » jclouds-chef #768](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-chef/768/) 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/jclo

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Reopened #284. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/284

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Closed #284. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/284

Re: [jclouds-chef] Adding 2009 to the project POM (#40)

Reopened #40. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-chef/pull/40

Re: [jclouds-chef] Adding 2009 to the project POM (#40)

Closed #40. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-chef/pull/40

Re: [jclouds-labs-openstack] Adds support for Rackspace Cloud Big Data (#78)

[jclouds-labs-openstack-pull-requests #161](https://jclouds.ci.cloudbees.com/job/jclouds-labs-openstack-pull-requests/161/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-openstack/pull/78#issuecomment-34828193

Re: [jclouds-labs-openstack] Adds support for Rackspace Cloud Big Data (#78)

[jclouds » jclouds-labs-openstack #839](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-labs-openstack/839/) 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/jcloud

Re: [jclouds] JCLOUDS-137: Retry on HTTP 500 AtmosError 1040 (#285)

> Committed to master and 1.7.x. Ah, I see my review is already a little late :-( @andrewgaul: most of the questions are minor, the only one that we may really need to look at, I think, is about making sure we increment the failure count correctly. --- Reply to this email directly or view it on

Re: [jclouds] JCLOUDS-137: Retry on HTTP 500 AtmosError 1040 (#285)

> + HttpCommand command = createMock(HttpCommand.class); > + String content = String.format(HTTP_MESSAGE_FORMAT, 1040, "The server > is busy. Please try again"); > + HttpResponse response = > HttpResponse.builder().statusCode(500).payload(content).build(); > + > + expect(comma

Re: [jclouds] JCLOUDS-137: Retry on HTTP 500 AtmosError 1040 (#285)

> + } > + > + @Inject(optional = true) > + @Named(Constants.PROPERTY_MAX_RETRIES) > + private int retryCountLimit = 5; > + @Resource > + protected Logger logger = Logger.NULL; > + > + public boolean shouldRetryRequest(HttpCommand command, HttpResponse > response) { > + if (comma

Re: [jclouds] JCLOUDS-137: Retry on HTTP 500 AtmosError 1040 (#285)

> + > + @Inject > + public AtmosServerErrorRetryHandler(BackoffLimitedRetryHandler > backoffHandler, > +AtmosUtils utils) { > + this.backoffHandler = backoffHandler; > + this.utils = utils; > + } > + > + @Inject(optional = true) > + @Named(Constants.PROPERTY_MAX_RET

Re: [jclouds] JCLOUDS-137: Retry on HTTP 500 AtmosError 1040 (#285)

> + return false; > + } > + if (response.getStatusCode() == 500) { > + byte[] content = > HttpUtils.closeClientButKeepContentStream(response); > + // Content can be null in the case of HEAD requests > + if (content != null) { > +try { > +

Re: [jclouds] JCLOUDS-137: Retry on HTTP 500 AtmosError 1040 (#285)

> + * @author Andrew Gaul > + */ > +public class AtmosServerErrorRetryHandler implements HttpRetryHandler { > + private final AtmosUtils utils; > + private final BackoffLimitedRetryHandler backoffHandler; > + > + @Inject > + public AtmosServerErrorRetryHandler(BackoffLimitedRetryHandler >

Build failed in Jenkins: jclouds-1.7.x-branch #42

See -- [...truncated 272 lines...] [INFO] jclouds openstack-cinder api .. SKIPPED [INFO] jclouds openstack-nova api SKIPPED [INFO] jclouds openstack-

Build failed in Jenkins: jclouds-1.7.x-branch » Apache jclouds Project #42

See -- maven3-agent.jar already up to date maven3-interceptor.jar already up to date maven3-interceptor-commons.jar already up to date <===[JENKINS REMOTING C

Jenkins build is back to normal : jclouds-1.7.x-branch » Apache jclouds Project #41

See

Build failed in Jenkins: jclouds-1.7.x-branch » jclouds Components Core #41

See -- [...truncated 3693 lines...] [WARNING]

Build failed in Jenkins: jclouds-1.7.x-branch #41

See -- [...truncated 4441 lines...] [INFO] jclouds netty payload module .. SKIPPED [INFO] jclouds enterprise Module . SKIPPED [INFO] jclouds Amazon Si

  1   2   >