Re: [jclouds] JCLOUDS-619 (#427)

2014-07-02 Thread mvrueden
@andrewgaul it works now. I rebased my commits against master. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/427#issuecomment-47872856

Jenkins build became unstable: jclouds » jclouds compute core #2664

2014-07-02 Thread jenkins-no-reply
See

Re: [jclouds-labs-google] Don't fail on tag filtering when group is not specified. (#26)

2014-07-02 Thread Andrew Phillips
> What is the status of this pull request? Let's ping @abayer one more time, otherwise I think we should be good to go. Could you kindly rebase the PR, by the way, because at present it doesn't look like it can be merged? Thanks for the ping, @najtmar! --- Reply to this email directly or view

Jenkins build is back to normal : jclouds » jclouds sts api #2662

2014-07-02 Thread jenkins-no-reply
See

Re: [jclouds] JCLOUDS-585 - Create HP Cloud block storage provider (#395)

2014-07-02 Thread Andrew Phillips
> I run live tests against all of my PRs after merging from master or 1.7.x > every few days until they are > merged Ah, great, thanks for explaining! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/395#issuecomment-47850511

Re: [jclouds-labs-aws] Default value for aws identity/credential property (#33)

2014-07-02 Thread Andrew Phillips
+1 - looks good to me too. Thanks, @rcoedo and @andrewgaul! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-aws/pull/33#issuecomment-47850588

Re: [jclouds] Default values for aws credential properties (#432)

2014-07-02 Thread Andrew Phillips
> jclouds-pull-requests #959 UNSTABLE Unrelated [test failure](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/org.apache.jclouds$jclouds-compute/959/testReport/junit/org.jclouds.compute.util/ConcurrentOpenSocketFinderTest/testChecksSocketsConcurrently/). Thanks, @rcoedo! --- Reply t

Re: [jclouds] Handle short reads in BasePayloadSlicer (d43c3ea)

2014-07-02 Thread Andrew Phillips
> Sorry I stopped short on fixing this but sometimes you have to cut your > losses. *grin*. Thanks for the fix - we'll tackle the rest when we get to it, I guess ;-) --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/commit/d43c3ea3e0e31663657bb8d99209a35

Re: [jclouds] Handle short reads in BasePayloadSlicer (d43c3ea)

2014-07-02 Thread Andrew Gaul
This code needs a lot more than style improvements: it arbitrarily buffers data in-memory and I do not believe its `InputStream` allows retries. Sorry I stopped short on fixing this but sometimes you have to cut your losses. --- Reply to this email directly or view it on GitHub: https://github.

Re: [jclouds] Default values for aws credential properties (#432)

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

Re: [jclouds-labs-google] JCLOUD-458:Added Bucket Operation with live tests (#31)

2014-07-02 Thread Andrew Phillips
> +BucketLifeCycle lifeCycle, StorageClass storageClass) { > + > + super(Kind.BUCKET, id, selfLink, etag); > + this.projectNumber = projectNumber; > + this.timeCreated = checkNotNull(timeCreated, "timeCreated"); > + this.metageneration = checkNotNull(metageneration,

Re: [jclouds-labs-google] JCLOUD-458:Added Bucket Operation with live tests (#31)

2014-07-02 Thread Andrew Phillips
> + private final String name; > + private final Long projectNumber; > + private final Date timeCreated; > + private final Long metageneration; > + private final Set acl; > + private final Set defaultObjectAcl; > + private final Owner owner; > + private final Location location; > +

Re: [jclouds] Default values for aws credential properties (#432)

2014-07-02 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #959](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/959/) 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/432#issuecomment-47847963

Re: [jclouds-labs-google] JCLOUD-458:Added Bucket Operation with live tests (#31)

2014-07-02 Thread Andrew Phillips
> +Long metageneration, Set acl, > Set defaultObjectAcl, > +Owner owner, Location location, Website website, Logging > logging, Versioning versioning, Set cors, > +BucketLifeCycle lifeCycle, StorageClass storageClass) { > + > + super(Kind.BUCKET, id, selfL

Re: [jclouds] Handle short reads in BasePayloadSlicer (d43c3ea)

2014-07-02 Thread Andrew Phillips
Just to ensure I'm getting this right: we accumulate the value of successive reads in the offset until read == -1. Then we either return null if offset == 0, else we create a payload. I.e. is this equivalent to: ``` int offset = 0; while ((read = input.read(content, offset, readLen - offset) != -

Re: [jclouds] Default values for aws credential properties (#432)

2014-07-02 Thread BuildHive
[jclouds » jclouds #1319](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1319/) 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/432#issuecomme

Re: [jclouds] JCLOUDS-619 (#427)

2014-07-02 Thread Andrew Phillips
Thanks for the updates, @mvrueden and the detailed review, @andrewgaul! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/427#issuecomment-47846953

Re: [jclouds] Default values for aws credential properties (#432)

2014-07-02 Thread Andrew Gaul
Pushed to master as 3932e8a3bd0a12085c214cf5f927a0830f757f0e. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/432#issuecomment-47846074

Re: [jclouds] Default values for aws credential properties (#432)

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

Re: [jclouds] JCLOUDS-619 (#427)

2014-07-02 Thread Andrew Gaul
@mvrueden `BasePayloadSlicer.getNextPayload` did not handle short reads correctly and I pushed a fix to master as d43c3ea3e0e31663657bb8d99209a3535b5bfcc6 and 1.7.x as c85d728a5a6e8bec3283703aa2fda22aef4dd925. Please rebase your commit against master and test again. --- Reply to this email di

Re: [jclouds-labs-aws] Default value for aws identity/credential property (#33)

2014-07-02 Thread Andrew Gaul
Pushed to master as c4dad4c42c89267aaa97c004e2961e00983969d8. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-aws/pull/33#issuecomment-47845275

Re: [jclouds] JCLOUDS-585 - Create HP Cloud block storage provider (#395)

2014-07-02 Thread Chris Custine
@demobox Absolutely... there are base live tests that are fairly thorough in the openstack-cinder project and the HP provider delegates to them. I run live tests against all of my PRs after merging from master or 1.7.x every few days until they are merged :-) --- Reply to this email directly

Re: [jclouds-labs-aws] Default value for aws identity/credential property (#33)

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

Re: [jclouds-labs-aws] Default value for aws identity/credential property (#33)

2014-07-02 Thread BuildHive
[jclouds » jclouds-labs-aws #1090](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-labs-aws/1090/) 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-

[jclouds] Default values for aws credential properties (#432)

2014-07-02 Thread Roman Coedo
Every project was using undefined properties in their pom.xml. A default value for test.aws.identity and test.aws.credential was added. You can merge this Pull Request by running: git pull https://github.com/rcoedo/jclouds credentials Or you can view, comment on it, or merge it online at: ht

Re: [jclouds-labs] [JCLOUDS-500] Initial commit for docker (#57)

2014-07-02 Thread Andrew Kennedy
> + > + if (templateOptions.getCommands().isPresent()) { > + containerConfigBuilder.cmd(templateOptions.getCommands().get()); > + } > + > + if (templateOptions.getMemory().isPresent()) { > + containerConfigBuilder.memory(templateOptions.getMemory().get()); > + }

Re: [jclouds] [JCLOUDS-474] refactor SoftLayer support (#431)

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

Jenkins build is back to normal : jclouds-labs-google #160

2014-07-02 Thread jenkins-no-reply
See

Jenkins build is back to normal : jclouds-labs-google » Apache jclouds labs google #160

2014-07-02 Thread jenkins-no-reply
See

Re: [jclouds-labs] [JCLOUDS-500] Initial commit for docker (#57)

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

Re: [jclouds] [JCLOUDS-474] refactor SoftLayer support (#431)

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

Re: [jclouds-labs-google] JCLOUD-458:Added Bucket Operation with live tests (#31)

2014-07-02 Thread Matt Stephenson
@hsbhathiya @demobox brings up some good minor points, please open a separate PR when you have time with these changes. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-google/pull/31#issuecomment-47840596

Re: [jclouds-labs-google] JCLOUD-458:Added Bucket Operation with live tests (#31)

2014-07-02 Thread Andrew Phillips
> +import > org.jclouds.googlecloudstorage.domain.DomainResourceRefferences.Location; > +import > org.jclouds.googlecloudstorage.domain.DomainResourceRefferences.StorageClass; > +import org.jclouds.googlecloudstorage.domain.internal.BucketCors; > +import org.jclouds.googlecloudstorage.domain.inte

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

2014-07-02 Thread CloudBees pull request builder plugin
[jclouds-labs-openstack-pull-requests #304](https://jclouds.ci.cloudbees.com/job/jclouds-labs-openstack-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-openstack/pull/111#issuecomment-4784

Re: [jclouds-labs] [JCLOUDS-500] Initial commit for docker (#57)

2014-07-02 Thread BuildHive
[jclouds » jclouds-labs #1258](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-labs/1258/) 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-labs/pul

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

2014-07-02 Thread BuildHive
[jclouds » jclouds-labs-openstack #1365](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-labs-openstack/1365/) 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/jclo

Jenkins build is back to normal : jclouds » jclouds-labs-openstack #1365

2014-07-02 Thread BuildHive
See

Re: [jclouds-chef] JCLOUDS-617: Use the configured JCE provider in the Cipher payloads (#45)

2014-07-02 Thread Andrew Phillips
+1 - looks good to me. Thanks, @nacx! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-chef/pull/45#issuecomment-47839514

Re: [jclouds-labs-aws] Default value for aws identity/credential property (#33)

2014-07-02 Thread Roman Coedo
Sure, I will. I will change the string to FIXME_IDENTITY/FIXME_CREDENTIAL here too. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-aws/pull/33#issuecomment-47839520

Re: [jclouds-labs-aws] Default value for aws identity/credential property (#33)

2014-07-02 Thread Andrew Gaul
@rcoedo Would you mind sending a pull request for this in jclouds core, at least for the Amazon providers? --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-aws/pull/33#issuecomment-47839214

Re: [jclouds] [JCLOUDS-474] refactor SoftLayer support (#431)

2014-07-02 Thread BuildHive
[jclouds » jclouds #1318](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1318/) 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/431#issuecomme

Re: [jclouds-labs-google] JCLOUD-458:Added Bucket Operation with live tests (#31)

2014-07-02 Thread Andrew Phillips
> @@ -44,4 +45,11 @@ > @Delegate > @Path("") > BucketAccessControlsApi getBucketAccessControlsApi(); > + > + /** > +* Provides access to Bucket features > +*/ > + @Delegate > + @Path("") > + BucketApi getBucketsApi(); Should this be `getBucketApi()`? --- Reply to

Re: [jclouds-labs-google] JCLOUD-458:Added Bucket Operation with live tests (#31)

2014-07-02 Thread Andrew Phillips
> @@ -111,6 +111,7 @@ > > ${jclouds.blobstore.httpstream.md5} > > ${test.google-cloud-storage.identity} > > ${test.google-cloud-storage.credential} > + > ${test.google-cloud-storage.project-number} >

Re: [jclouds-labs] [JCLOUDS-500] Initial commit for docker (#57)

2014-07-02 Thread Andrea Turli
> + return this; > + } > + > + public TemplateOptions commands(Iterable commands) { > + for (String command : checkNotNull(commands, "commands")) > + checkNotNull(command, "all commands must be non-empty"); > + this.commands = Optional.> > of(ImmutableList.copyOf(command

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

2014-07-02 Thread Zack Shoylev
Hi reviewers! There were a number of concerns with how the domain objects were implemented in the old neutron and some different concerns with the refactored code. Problems with the old code: Options and Mapbinders (extra code, doesn't utilize GSON enough). Newer code: Create/Update/List objects

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

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

Re: [jclouds-labs-google] JCLOUD-458:Added Bucket Operation with live tests (#31)

2014-07-02 Thread Matt Stephenson
@andrewgaul I recalled that about a day ago when @hsbhathiya informed me his commit hadn't made it. I got the internal bidirectional github branch tracking scripts I setup at Google confused with those at Apache. I thought we had bidirectional sync, I'll work with infra to see if I can improve

Re: [jclouds-labs] [JCLOUDS-500] Initial commit for docker (#57)

2014-07-02 Thread Andrew Kennedy
> + > + if (templateOptions.getCommands().isPresent()) { > + containerConfigBuilder.cmd(templateOptions.getCommands().get()); > + } > + > + if (templateOptions.getMemory().isPresent()) { > + containerConfigBuilder.memory(templateOptions.getMemory().get()); > + }

Build failed in Jenkins: jclouds » jclouds-labs-openstack #1364

2014-07-02 Thread BuildHive
See Changes: [Zack Shoylev] Neutron Refactoring [Zack Shoylev] Adds code to test for AddressPairs [Zack Shoylev] Changed order of fields [Zack Shoylev] Reworks how builder works for AddressPair. (experimenta

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

2014-07-02 Thread BuildHive
[jclouds » jclouds-labs-openstack #1364](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-labs-openstack/1364/) 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:

Build failed in Jenkins: jclouds #2661

2014-07-02 Thread jenkins-no-reply
See -- [...truncated 7582 lines...] [Unloading class sun.reflect.GeneratedConstructorAccessor16] [Unloading class GregorSamsa] [Unloading class sun.reflect.GeneratedSerializationConstructorAccessor65] [Unl

Build failed in Jenkins: jclouds » jclouds sts api #2661

2014-07-02 Thread jenkins-no-reply
See -- [...truncated 97 lines...] Starting test test400WithInUseSetsIllegalStateException(org.jclouds.aws.handlers.ParseAWSErrorFromXmlContentTest) [pool-6-thread-2] Test test

[jira] [Commented] (JCLOUDS-458) Google Cloud Storage support

2014-07-02 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-458?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14050702#comment-14050702 ] ASF subversion and git services commented on JCLOUDS-458: - Commit

Re: [jclouds-labs-google] JCLOUD-458:Added Bucket Operation with live tests (#31)

2014-07-02 Thread Andrew Gaul
@mattstep Note that merging via GitHub does *not* push to the official Apache git repository; jclouds-mirror pushes from Apache to GitHub. Committers must git push to the Apache repository. I reworded the commit message, made some whitespace changes, and pushed to master as b2d343d06a8275d0b9

Re: [jclouds-labs] [JCLOUDS-500] Initial commit for docker (#57)

2014-07-02 Thread Andrew Kennedy
> + return this; > + } > + > + public TemplateOptions commands(Iterable commands) { > + for (String command : checkNotNull(commands, "commands")) > + checkNotNull(command, "all commands must be non-empty"); > + this.commands = Optional.> > of(ImmutableList.copyOf(command

Re: [jclouds] Updates Nova default API to v2 (#390)

2014-07-02 Thread Jeremy Daggett
@ccustine Thanks! The rebase/merge is going to be "fun" indeed! It might take me some time to get through it, but I wanted to make sure that all of your Nova changes were merged prior to this PR. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/390#i

Re: [jclouds] Refactor SoftLayer CCI support (#296)

2014-07-02 Thread Andrea Turli
closing this PR as it has been re-opened against master at https://github.com/jclouds/jclouds/pull/431 --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/296#issuecomment-47831666

Re: [jclouds] Refactor SoftLayer CCI support (#296)

2014-07-02 Thread Andrea Turli
Closed #296. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/296#event-137605769

[jclouds] [JCLOUDS-474] refactor SoftLayer support (#431)

2014-07-02 Thread Andrea Turli
It modernizes SoftLayer CCI support by re-using the same API calls used by SoftLayer Python client. The implementation should be not more easily maintainable and robust. You can merge this Pull Request by running: git pull https://github.com/andreaturli/jclouds new/softlayer-cci Or you can vie

Re: [jclouds] openstack console (#339)

2014-07-02 Thread pimenas
:) You're all very welcome and thank you also!!! It's been a very learning experience for me. @everett-toews indeed its [me](https://twitter.com/pimenas) on twitter :) --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/339#issuecomment-47831452

Re: [jclouds-labs-openstack] Rackspace Auto Scale Provider Update (#112)

2014-07-02 Thread Chris Custine
+1 Looks good. I do agree that metadata probably doesn't need to be versioned so I think that is a good future opportunity. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-openstack/pull/112#issuecomment-47829287

Jenkins build is back to normal : jclouds » jclouds-labs-google #1082

2014-07-02 Thread BuildHive
See

Re: [jclouds] Updates Nova default API to v2 (#390)

2014-07-02 Thread Chris Custine
+1 This looks good to me and it is certainly time to update these defaults. @jdaggett you might have some rebasing/merging issues after the HP Cloud PRs because I also updated some of that json test data. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/p

Re: [jclouds] openstack console (#339)

2014-07-02 Thread Everett Toews
EPIC pull request for your first contribution to jclouds. Thank you so much for sticking with it! Is this [you on Twitter](https://twitter.com/pimenas)? We'd love to give you a shoutout for the work you've done here. --- Reply to this email directly or view it on GitHub: https://github.com/jclo

Re: [jclouds] openstack console (#339)

2014-07-02 Thread Jeremy Daggett
Closed #339. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/339#event-137559866

Re: [jclouds] openstack console (#339)

2014-07-02 Thread Jeremy Daggett
@pimenas Thanks for the contribution! Merged to master `b259af4` and 1.7.x `10a1b23` --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/339#issuecomment-47819181

[jira] [Updated] (JCLOUDS-585) Create HP Cloud block storage provider

2014-07-02 Thread Andrew Phillips (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-585?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andrew Phillips updated JCLOUDS-585: Fix Version/s: 1.7.4 > Create HP Cloud block storage provider > --

Re: [jclouds] JCLOUDS-585 - Create HP Cloud block storage provider (#395)

2014-07-02 Thread Andrew Phillips
Thanks, @ccustine! Out of curiosity: do we have any live tests results for this? --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/395#issuecomment-47816273

Re: [jclouds] JCLOUDS-585 - Create HP Cloud block storage provider (#395)

2014-07-02 Thread Jeremy Daggett
Pushed to 1.7.x `fb373d4` --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/395#issuecomment-47813868

[jira] [Commented] (JCLOUDS-585) Create HP Cloud block storage provider

2014-07-02 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-585?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14050499#comment-14050499 ] ASF subversion and git services commented on JCLOUDS-585: - Commit

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

2014-07-02 Thread Christopher Dancy
Let's leave it open a while longer to see what others have to say. In testing the 'create()' method is just making unnecessary http calls and filling up the logs with the above. If we can agree on the above, meaning to deprecate, or at the least don't use it at all, then I'll create a second PR

Re: [jclouds] Added the CONTRIBUTING file (#430)

2014-07-02 Thread Andrew Phillips
> jclouds » jclouds #1315 FAILURE > jclouds-pull-requests #957 FAILURE > jclouds-java-7-pull-requests #1428 FAILURE Tsk. Tsk ;-) At least our CI system seems to be working reasonably, for now. Thanks for the fix, @nacx! --- Reply to this email directly or view it on GitHub: https://github.com/jc

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

2014-07-02 Thread Andrew Phillips
> Upon further thought I wonder if we should remove/deprecate the call to > "create()" altogether? I'm not familiar enough with this part of the code to comment - perhaps @zack-shoylev, @jdaggett or @everett-toews have some thoughts here? In the meantime, would you prefer to leave this PR open

Jenkins build is back to stable : jclouds » jclouds compute core #2659

2014-07-02 Thread jenkins-no-reply
See

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

2014-07-02 Thread Christopher Dancy
Upon further thought I wonder if we should remove/deprecate the call to "create()" altogether? Searching the internets I'm not the first to come across this issue openstack. The "create()" call fails because it's not passing a pool-name which I assume is why the "allocate(String poolName)" call

Jenkins build is back to normal : jclouds » jclouds-labs-openstack #1362

2014-07-02 Thread BuildHive
See

Jenkins build is back to normal : jclouds » jclouds-karaf #1098

2014-07-02 Thread BuildHive
See

Build failed in Jenkins: jclouds » jclouds-labs-google #1081

2014-07-02 Thread BuildHive
See -- Started by upstream project "jclouds/jclouds" build number 1316 originally caused by: Started by GitHub push by jclouds-mirror Building remotely on s-6a190576 in workspace <

Jenkins build became unstable: jclouds » jclouds compute core #2658

2014-07-02 Thread jenkins-no-reply
See

Jenkins build is back to normal : jclouds » jclouds #1316

2014-07-02 Thread BuildHive
See

Re: [jclouds-labs-openstack] Rackspace Auto Scale Provider Update (#112)

2014-07-02 Thread CloudBees pull request builder plugin
[jclouds-labs-openstack-pull-requests #302](https://jclouds.ci.cloudbees.com/job/jclouds-labs-openstack-pull-requests/302/) 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-labs-openstack/pull/11

Build failed in Jenkins: jclouds » jclouds-labs-openstack #1361

2014-07-02 Thread BuildHive
See Changes: [jeremy.daggett] Add Rackspace Auto Scale UK provider and update US regions [jeremy.daggett] Removed version from package name in Rackspace Auto Scale US provider [jeremy.daggett] Updated failin

Re: [jclouds-labs-openstack] Rackspace Auto Scale Provider Update (#112)

2014-07-02 Thread BuildHive
[jclouds » jclouds-labs-openstack #1361](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-labs-openstack/1361/) 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:

Re: [jclouds-labs-openstack] Rackspace Auto Scale Provider Update (#112)

2014-07-02 Thread Jeremy Daggett
@nacx Thanks, it totally makes sense to me now! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-openstack/pull/112#issuecomment-47799431

Re: [jclouds-labs-openstack] Rackspace Auto Scale Provider Update (#112)

2014-07-02 Thread Ignasi Barrera
In environments where the ServiceLoader might be different (I recall issues with this on Android), users might want to directly pass a metadata instance to the ContextBuilder to avoid the ServiceLoader lookup. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclo

Re: [jclouds-labs-openstack] Rackspace Auto Scale Provider Update (#112)

2014-07-02 Thread Jeremy Daggett
Hmm, I can't understand how/why anyone would ever import or instantiate this class directly. AutoscaleUSProviderMetadata is an internal class and only used by the Service Loader (and expect tests). ?? I reverted the commit that changed the package name and we can change it in a future release.

Jenkins build is back to normal : jclouds-labs » Apache jclouds labs #507

2014-07-02 Thread jenkins-no-reply
See

Jenkins build is back to normal : jclouds-chef #854

2014-07-02 Thread jenkins-no-reply
See

Jenkins build is back to normal : jclouds-chef » Apache jclouds Chef :: Core #854

2014-07-02 Thread jenkins-no-reply
See

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

2014-07-02 Thread BuildHive
See

Jenkins build is back to normal : jclouds » jclouds-labs #1255

2014-07-02 Thread BuildHive
See

Jenkins build is back to normal : jclouds-labs-aws #134

2014-07-02 Thread jenkins-no-reply
See

Jenkins build is back to normal : jclouds-labs-aws » Apache jclouds labs aws #134

2014-07-02 Thread jenkins-no-reply
See

Re: [jclouds] Added the CONTRIBUTING file (#430)

2014-07-02 Thread Ignasi Barrera
The file has already been excluded from the RAT check to fix the failure. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/430#issuecomment-47795505

Jenkins build is back to normal : jclouds-labs-aws #134

2014-07-02 Thread jenkins-no-reply
See

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

2014-07-02 Thread BuildHive
See Changes: [Ignasi Barrera] Exclude the CONTRIBUTING file from the rat check -- [...truncated 26 lines...] [WARNING] [INFO] -

Re: [jclouds-chef] JCLOUDS-617: Use the configured JCE provider in the Cipher payloads (#45)

2014-07-02 Thread Ignasi Barrera
Closed #45. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-chef/pull/45#event-137479099

Re: [jclouds-chef] JCLOUDS-617: Use the configured JCE provider in the Cipher payloads (#45)

2014-07-02 Thread Ignasi Barrera
Pushed to [master](https://git-wip-us.apache.org/repos/asf?p=jclouds-chef.git;a=commit;h=7d6e1fdd5379755b6d15eec5e34374253384c572). --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-chef/pull/45#issuecomment-47795278

[jira] [Commented] (JCLOUDS-617) Unable to use Chef API with JCE provider with default RSA transformation padding other than PKCS1

2014-07-02 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-617?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14050236#comment-14050236 ] ASF subversion and git services commented on JCLOUDS-617: - Commit

Jenkins build is back to normal : jclouds » jclouds-labs-aws #1086

2014-07-02 Thread BuildHive
See

  1   2   >