Re: [jclouds-labs-openstack] The create and stream methods in the Message API for OpenStack Marconi. (#46)

2013-11-15 Thread Ignasi Barrera
Indeed! :) --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-openstack/pull/46#issuecomment-28621429

RE: Project Lombok - Builders (and other annotations)

2013-11-15 Thread Adrian Cole
Pretty much all of these require that your ide compiles with their jar in the classpath as a provided dep. Some IDEs also require explicit enabling of annotation processing. Maven integration is pretty tight wrt this in IDEs I've used. When you change a source file, it is processed with auto or

Re: Project Lombok - Builders (and other annotations)

2013-11-15 Thread Adrian Cole
Ps Auto Value (currently) uses asm for bytecode manipulation as opposed to Auto Factory, which writes generated source files (via javawriter). Makes it more magicy, but not necessarily bad. -A On Nov 14, 2013 10:42 PM, "Andrew Gaul" wrote: > On Mon, Nov 11, 2013 at 11:11:35AM -0800, Andrew Gaul

Re: [jclouds-site] JCLOUDS-215 Adds autoscale documentation. (#32)

2013-11-15 Thread Zack Shoylev
> +1. Open Constants.java for editing. > +1. Go to the example code > [Constants.java](https://github.com/jclouds/jclouds-examples/blob/master/rackspace/src/main/java/org/jclouds/examples/rackspace/autoscale/Constants.java), > read it over, and copy the code into your file. > + > +### Compile and

Re: [jclouds-site] JCLOUDS-215 Adds autoscale documentation. (#32)

2013-11-15 Thread Andrew Phillips
> +1. Open Constants.java for editing. > +1. Go to the example code > [Constants.java](https://github.com/jclouds/jclouds-examples/blob/master/rackspace/src/main/java/org/jclouds/examples/rackspace/autoscale/Constants.java), > read it over, and copy the code into your file. > + > +### Compile and

Re: [jclouds-site] JCLOUDS-215 Adds autoscale documentation. (#32)

2013-11-15 Thread Andrew Phillips
> +1. You can find these APIs in the latest > [Javadoc](http://demobox.github.com/jclouds-maven-site/latest/apidocs). > + > +### The Source Code > + > +1. Create the directory hierarchy > org/jclouds/examples/rackspace/clouddatabases/ in your jclouds directory. > +1. Create Java source files call

Re: [jclouds-site] JCLOUDS-215 Adds autoscale documentation. (#32)

2013-11-15 Thread Andrew Phillips
Thanks for the updates! Something weird going on with the numbering now: ![image](https://f.cloud.github.com/assets/223702/1555039/26c02824-4e53-11e3-9772-34cee2691214.png) --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-site/pull/32#issuecomment-28613914

Re: [jclouds] JCLOUDS-381. Allow explicit naming of nodes via TemplateOptions. (#206)

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

Re: [jclouds] JCLOUDS-381. Allow explicit naming of nodes via TemplateOptions. (#206)

2013-11-15 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #408](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/408/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/206#issuecomment-28610325

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

2013-11-15 Thread BuildHive
See

Re: [jclouds] JCLOUDS-381. Allow explicit naming of nodes via TemplateOptions. (#206)

2013-11-15 Thread Andrew Bayer
Ok, actually, turned out to be less than an hour of work. woo? Softlayer still only supports old groups, but every other provider in core should be good. I've done live tests against EC2, HP Cloud, and (with a yet-to-be-pushed branch) GCE. --- Reply to this email directly or view it on GitHub: h

Re: [jclouds-labs-openstack] The create and stream methods in the Message API for OpenStack Marconi. (#46)

2013-11-15 Thread Andrew Phillips
Thanks for your review patience, @everett-toews! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-openstack/pull/46#issuecomment-28606163

Re: [jclouds-labs-openstack] The create and stream methods in the Message API for OpenStack Marconi. (#46)

2013-11-15 Thread Andrew Phillips
> > - protected QueueStats(MessagesStats messages) { > - this.messages = messages; > + protected QueueStats(MessagesStats messageStats) { > + this.messages = checkNotNull(messageStats); [minor] `checkNotNull(messageStats, "messageStats required");`? --- Reply to this email directl

Jenkins build is back to normal : jclouds #2396

2013-11-15 Thread jenkins-no-reply
See

Jenkins build is back to normal : jclouds » jclouds loadbalancer core #2396

2013-11-15 Thread jenkins-no-reply
See

Build failed in Jenkins: jclouds » jclouds #628

2013-11-15 Thread BuildHive
See -- [...truncated 20854 lines...] at java.net.URLClassLoader.findClass(URLClassLoader.java:354) at java.lang.ClassLoader.loadClass(ClassLoader.java:424) at sun.misc.Lau

Re: [jclouds] JCLOUDS-383. Adding c3 instance sizes. (#208)

2013-11-15 Thread BuildHive
[jclouds » jclouds #628](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/628/) 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/jcloud

Re: [jclouds] JCLOUDS-383. Adding c3 instance sizes. (#208)

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

Re: [jclouds] JCLOUDS-383. Adding c3 instance sizes. (#208)

2013-11-15 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #407](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/407/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/208#issuecomment-28604696

Re: [jclouds] JCLOUDS-381. Allow explicit naming of nodes via TemplateOptions. (#206)

2013-11-15 Thread Andrew Bayer
Yup, looks like I need to add logic to every API/provider to save the group in instance metadata or somewhere similar, which is gonna necessitate a fair amount of work. I think I can probably get it done this weekend, though. --- Reply to this email directly or view it on GitHub: https://github.

Re: [jclouds] JCLOUDS-381. Allow explicit naming of nodes via TemplateOptions. (#206)

2013-11-15 Thread Andrew Bayer
Live tests are coming together, but not quite there yet - some problems in the tests themselves, but also the realization that marking nodes as in a group becomes more problematic when you can't necessarily rely on the instance name to figure out what "group" they're in. I'm playing around with

RE: Project Lombok - Builders (and other annotations)

2013-11-15 Thread Zack Shoylev
I didn't check - how's the IDE situation with auto? I might find the time to add it in some refactorings I have to do anyways. From: Adrian Cole [adrian.f.c...@gmail.com] Sent: Friday, November 15, 2013 11:58 AM To: dev@jclouds.apache.org Subject: Re: Proje

Re: [jclouds] JCLOUDS-381. Allow explicit naming of nodes via TemplateOptions. (#206)

2013-11-15 Thread Andrew Bayer
Working on live tests now - trying to figure out whether the failures I'm seeing are just erratic. I'm tending to think they're not consistently reproducible, but I'm not 100% sure. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/206#issuecomment-28

Re: [jclouds] JCLODUS-383. Adding c3 instance sizes. (#208)

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

Re: [jclouds] JCLODUS-383. Adding c3 instance sizes. (#208)

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

Re: [jclouds] JCLODUS-383. Adding c3 instance sizes. (#208)

2013-11-15 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #406](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/406/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/208#issuecomment-28592639

Re: [jclouds] JCLODUS-383. Adding c3 instance sizes. (#208)

2013-11-15 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #405](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/405/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/208#issuecomment-28592638

Re: [jclouds] JCLOUDS-381. Allow explicit naming of nodes via TemplateOptions. (#206)

2013-11-15 Thread BuildHive
[jclouds » jclouds #626](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/626/) 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/206#issuecomment

[jclouds] JCLODUS-383. Adding c3 instance sizes. (#209)

2013-11-15 Thread Andrew Bayer
You can merge this Pull Request by running: git pull https://github.com/abayer/jclouds-1 jclouds-383-1.6.x Or you can view, comment on it, or merge it online at: https://github.com/jclouds/jclouds/pull/209 -- Commit Summary -- * JCLODUS-383. Adding c3 instance sizes. -- File Changes --

[jclouds] JCLODUS-383. Adding c3 instance sizes. (#208)

2013-11-15 Thread Andrew Bayer
You can merge this Pull Request by running: git pull https://github.com/abayer/jclouds-1 jclouds-383 Or you can view, comment on it, or merge it online at: https://github.com/jclouds/jclouds/pull/208 -- Commit Summary -- * JCLODUS-383. Adding c3 instance sizes. -- File Changes --

Re: Project Lombok - Builders (and other annotations)

2013-11-15 Thread Adrian Cole
FWIW, I plan to help with auto anyway. If you have problems trying it out, cc me on the github issue. -A On Nov 14, 2013 10:42 PM, "Andrew Gaul" wrote: > On Mon, Nov 11, 2013 at 11:11:35AM -0800, Andrew Gaul wrote: > > On Mon, Nov 11, 2013 at 10:36:26AM -0800, Jeremy Daggett wrote: > > > Hi dev

Re: restarting the 1.6.3 release process on Friday

2013-11-15 Thread Andrew Bayer
Andrew, wait to kick off the 1.6.3 release process 'til I have a patch for https://issues.apache.org/jira/browse/JCLOUDS-383 if that's alright - it should be a trivial change, and it'd very good to have in there... I'll have a patch by noon. A. On Thu, Nov 14, 2013 at 5:15 AM, Andrea Turli wrote

Re: [jclouds] JCLOUDS-381. Allow explicit naming of nodes via TemplateOptions. (#206)

2013-11-15 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #404](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/404/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/206#issuecomment-28587935

Re: [jclouds] JCLOUDS-381. Allow explicit naming of nodes via TemplateOptions. (#206)

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

Re: [jclouds] JCLOUDS-381. Allow explicit naming of nodes via TemplateOptions. (#206)

2013-11-15 Thread Andrew Bayer
Tests will pass now - the change that caused the tests to fail was actually me screwing up and deleting something, not the intended changes. Woopsie. About to try live tests! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/206#issuecomment-28586013

Re: [jclouds] JCLOUDS-381. Allow explicit naming of nodes via TemplateOptions. (#206)

2013-11-15 Thread Andrew Bayer
> @@ -353,7 +355,7 @@ public void testCreateTwoNodesWithRunScript() throws > Exception { >NodeMetadata node2 = nodes.last(); >// credentials aren't always the same >// assertEquals(node1.getCredentials(), node2.getCredentials()); > - > + System.err.println("node 1 baby

Build failed in Jenkins: jclouds #2395

2013-11-15 Thread jenkins-no-reply
See Changes: [Everett Toews] Style fixes and corrections. -- [...truncated 10076 lines...] [Unloading class GregorSamsa$1] [Unloading class GregorSamsa] [Unloading class GregorSamsa] [Unloading cl

Build failed in Jenkins: jclouds » jclouds loadbalancer core #2395

2013-11-15 Thread jenkins-no-reply
See -- Nov 15, 2013 12:09:32 PM org.apache.maven.cli.event.ExecutionEventLogger projectStarted INFO:

Re: 1.7.0 scheduling

2013-11-15 Thread Alex Heneveld
+1 Best, Alex On Nov 15, 2013 4:22 PM, "Everett Toews" wrote: > On Nov 14, 2013, at 12:18 PM, Andrew Bayer wrote: > > > So with 1.6.3 aiming to be out next week, I'd like to put a target date > on > > 1.7.0 - we really need to get that sucker out! I'm gonna tentatively plan > > to do the first R

Re: [jclouds-labs-openstack] The create and stream methods in the Message API for OpenStack Marconi. (#46)

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

Re: 1.7.0 scheduling

2013-11-15 Thread Everett Toews
On Nov 14, 2013, at 12:18 PM, Andrew Bayer wrote: > So with 1.6.3 aiming to be out next week, I'd like to put a target date on > 1.7.0 - we really need to get that sucker out! I'm gonna tentatively plan > to do the first RC on Monday, December 9th. +1 Everett

RE: Project Lombok - Builders (and other annotations)

2013-11-15 Thread Zack Shoylev
They have good documentation... From: Andrew Gaul [g...@apache.org] Sent: Friday, November 15, 2013 12:42 AM To: dev@jclouds.apache.org Subject: Re: Project Lombok - Builders (and other annotations) On Mon, Nov 11, 2013 at 11:11:35AM -0800, Andrew Gaul wrot

Re: [jclouds] applies tidies for imageChooser (JCLOUDS-331) as suggested in pull #202 (#207)

2013-11-15 Thread BuildHive
[jclouds » jclouds #624](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/624/) 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/207#issuecomment

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

2013-11-15 Thread BuildHive
See

Re: [jclouds] applies tidies for imageChooser (JCLOUDS-331) as suggested in pull #202 (#207)

2013-11-15 Thread BuildHive
[jclouds » jclouds #623](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/623/) 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

Re: [jclouds] applies tidies for imageChooser (JCLOUDS-331) as suggested in pull #202 (#207)

2013-11-15 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #403](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/403/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/207#issuecomment-28557815

Re: [jclouds] applies tidies for imageChooser (JCLOUDS-331) as suggested in pull #202 (#207)

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

Re: [jclouds] applies tidies for imageChooser (JCLOUDS-331) as suggested in pull #202 (#207)

2013-11-15 Thread CloudBees pull request builder plugin
[jclouds-java-7-pull-requests #865](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/865/) 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/207#issuecomment-28557627

Re: [jclouds] applies tidies for imageChooser (JCLOUDS-331) as suggested in pull #202 (#207)

2013-11-15 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #402](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/402/) 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/207#issuecomment-28557625

Re: [jclouds] jclouds-331 - imageChooser function backport (#202)

2013-11-15 Thread ahgittin
cool. done in #207. what command were you running when you got the whitespace warnings? happy to clean those up also. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/202#issuecomment-28556865

[jclouds] applies tidies for imageChooser (JCLOUDS-331) as suggested in pull #202 (#207)

2013-11-15 Thread ahgittin
good suggestions in #202 @demobox, thanks You can merge this Pull Request by running: git pull https://github.com/ahgittin/jclouds imageChooser-tidy Or you can view, comment on it, or merge it online at: https://github.com/jclouds/jclouds/pull/207 -- Commit Summary -- * applies tidies fo

Re: [jclouds] JCLOUDS-381. Allow explicit naming of nodes via TemplateOptions. (#206)

2013-11-15 Thread Ignasi Barrera
I've had a look and looks good to me. I'd be extremely happy to have this merged once tests pass! (I can only vote once in jira, though :D) --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/206#issuecomment-28552805