Re: [jclouds] JCLOUDS-467. Properly iterate over node names for EC2 instance creation. (#292)

2014-02-14 Thread BuildHive
[jclouds » jclouds #846](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/846/) 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/292#issuecomment

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

2014-02-14 Thread BuildHive
See

Re: [jclouds-labs-openstack] Adds support for multi-region implementation of Rackspace Cloud Files API (#79)

2014-02-14 Thread Zack Shoylev
Lots of refactoring as well, nice! It seems some of the javadoc has to be finished. I would say this is not a problem, especially if waiting for some service-side answers? --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-openstack/pull/79#issuecommen

Re: [jclouds-labs-openstack] Adds support for multi-region implementation of Rackspace Cloud Files API (#79)

2014-02-14 Thread Jeremy Daggett
> +* @param containerName > +* corresponds to {@link Container#name()}. > +* @return the CDNContainer or null, if not found. > +* > +* @see +* > href="http://docs.openstack.org/api/openstack-object-storage/1.0/content/retrieve-Container-metadata.html";> > +

Re: [jclouds-labs-openstack] Adds support for multi-region implementation of Rackspace Cloud Files API (#79)

2014-02-14 Thread Jeremy Daggett
> @@ -25,10 +25,10 @@ > > @Override > public Account apply(HttpResponse from) { > - return Account.builder() // > - > .bytesUsed(Long.parseLong(from.getFirstHeaderOrNull("X-Account-Bytes-Used"))) > // > - > .containerCount(Long.parseLong(from.getFirstHeaderOr

Re: [jclouds] Add keystone user add and delete methods. (#290)

2014-02-14 Thread Andrew Phillips
> jclouds » jclouds #845 UNSTABLE "Real" [test failures](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/org.apache.jclouds.api$openstack-keystone/845/testReport/junit/org.jclouds.openstack.keystone.v2_0.features/UserApiExpectTest/testCreateUser/) here. --- Reply to this email directly

Re: [jclouds-labs-openstack] Adds support for multi-region implementation of Rackspace Cloud Files API (#79)

2014-02-14 Thread Zack Shoylev
> +* @param containerName > +* corresponds to {@link Container#name()}. > +* @return the CDNContainer or null, if not found. > +* > +* @see +* > href="http://docs.openstack.org/api/openstack-object-storage/1.0/content/retrieve-Container-metadata.html";> > +

Re: [jclouds-labs-openstack] Adds support for multi-region implementation of Rackspace Cloud Files API (#79)

2014-02-14 Thread Zack Shoylev
> @@ -25,10 +25,10 @@ > > @Override > public Account apply(HttpResponse from) { > - return Account.builder() // > - > .bytesUsed(Long.parseLong(from.getFirstHeaderOrNull("X-Account-Bytes-Used"))) > // > - > .containerCount(Long.parseLong(from.getFirstHeaderOr

Jenkins build became unstable: jclouds » jclouds #845

2014-02-14 Thread BuildHive
See

Re: [jclouds] Add keystone user add and delete methods. (#290)

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

[jira] [Commented] (JCLOUDS-470) region missing from group name in AWSEC2SecurityGroupExtension

2014-02-14 Thread Andrew Bayer (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-470?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13902079#comment-13902079 ] Andrew Bayer commented on JCLOUDS-470: -- Pushed to master, will push to 1.7.x when bu

[jira] [Updated] (JCLOUDS-467) nameNodes only taking first name with AWS

2014-02-14 Thread Andrew Bayer (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-467?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andrew Bayer updated JCLOUDS-467: - Fix Version/s: 1.7.2 Went with the list approach - committed to master, PR pending BuildHive and

Re: [jclouds] Backport JCLOUDS-470 to 1.7.x (#294)

2014-02-14 Thread Andrew Phillips
Pending PR builders, +1 - looks good to me! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/294#issuecomment-35133169

Re: [jclouds] Backport JCLOUDS-467 to 1.7.x (#293)

2014-02-14 Thread Andrew Phillips
Pending PR builders, +1 - looks good to me! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/293#issuecomment-35133194

[jira] [Commented] (JCLOUDS-467) nameNodes only taking first name with AWS

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

[jclouds] Backport JCLOUDS-467 to 1.7.x (#293)

2014-02-14 Thread Andrew Bayer
You can merge this Pull Request by running: git pull https://github.com/abayer/jclouds-1 jclouds-467-for-1.7.x Or you can view, comment on it, or merge it online at: https://github.com/jclouds/jclouds/pull/293 -- Commit Summary -- * JCLOUDS-467. Properly iterate over node names for EC2

[jclouds] Backport JCLOUDS-470 to 1.7.x (#294)

2014-02-14 Thread Andrew Bayer
You can merge this Pull Request by running: git pull https://github.com/abayer/jclouds-1 jclouds-470-for-1.7.x Or you can view, comment on it, or merge it online at: https://github.com/jclouds/jclouds/pull/294 -- Commit Summary -- * JCLOUDS-470. Include region in group name for AWSEC2Se

[jira] [Commented] (JCLOUDS-470) region missing from group name in AWSEC2SecurityGroupExtension

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

Re: [jclouds-labs-openstack] Adds support for multi-region implementation of Rackspace Cloud Files API (#79)

2014-02-14 Thread Jeremy Daggett
@demobox I updated the PR title and additional clarification on running live tests. Thx! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-openstack/pull/79#issuecomment-35132057

Re: [jclouds] JCLOUDS-467. Properly iterate over node names for EC2 instance creation. (#292)

2014-02-14 Thread Andrew Phillips
A question and a minor comment, that's all. +1 - looks good to me. Thanks, @abayer! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/292#issuecomment-35131001

Re: [jclouds-labs-openstack] Adds support for Rackspace Cloud Files API (#79)

2014-02-14 Thread Andrew Phillips
@jdaggett Just to make sure I'm getting this right...this is a different/new version of the CloudFiles API as that covered by the [current provider](https://github.com/jclouds/jclouds/blob/master/apis/cloudfiles/src/main/java/org/jclouds/cloudfiles/CloudFilesApiMetadata.java)? If so, could you a

Re: [jclouds] JCLOUDS-467. Properly iterate over node names for EC2 instance creation. (#292)

2014-02-14 Thread Andrew Bayer
>if (generateInstanceNames && !common.containsKey("Name")) { > for (Map.Entry entry : > instancesById.entrySet()) { > String id = entry.getKey(); > String name; > -if (!nodeNames.isEmpty()) { > - name = Iterables.get(nodeNames,

Re: [jclouds] JCLOUDS-467. Properly iterate over node names for EC2 instance creation. (#292)

2014-02-14 Thread Andrew Phillips
>if (generateInstanceNames && !common.containsKey("Name")) { > for (Map.Entry entry : > instancesById.entrySet()) { > String id = entry.getKey(); > String name; > -if (!nodeNames.isEmpty()) { > - name = Iterables.get(nodeNames,

Re: [jclouds] JCLOUDS-467. Properly iterate over node names for EC2 instance creation. (#292)

2014-02-14 Thread Andrew Bayer
> @@ -175,12 +175,13 @@ public String apply(NodeMetadata in) { >Map instancesById = > Maps.uniqueIndex(input, instanceId); >ImmutableSet.Builder builder = > ImmutableSet. builder(); > > + List namesToUse = Lists.newArrayList(nodeNames); okiedokie! --- Reply to this email

Re: [jclouds] JCLOUDS-470. Include region in group name for AWSEC2SecurityGroupExtensi... (#291)

2014-02-14 Thread Andrew Bayer
> @@ -41,13 +46,6 @@ > import org.jclouds.net.domain.IpPermission; > import org.jclouds.net.domain.IpProtocol; > > -import com.google.common.base.Function; > -import com.google.common.base.Supplier; > -import com.google.common.cache.LoadingCache; > -import com.google.common.collect.Iterables; >

Re: [jclouds] JCLOUDS-467. Properly iterate over node names for EC2 instance creation. (#292)

2014-02-14 Thread Andrew Phillips
> @@ -175,12 +175,13 @@ public String apply(NodeMetadata in) { >Map instancesById = > Maps.uniqueIndex(input, instanceId); >ImmutableSet.Builder builder = > ImmutableSet. builder(); > > + List namesToUse = Lists.newArrayList(nodeNames); [minor] Static import of `Lists.newA

Re: [jclouds] JCLOUDS-470. Include region in group name for AWSEC2SecurityGroupExtensi... (#291)

2014-02-14 Thread Andrew Phillips
> jclouds-java-7-pull-requests #1073 UNSTABLE Unrelated [test failure](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/org.apache.jclouds$jclouds-compute/1073/testReport/junit/org.jclouds.compute.util/ConcurrentOpenSocketFinderTest/testChecksSocketsConcurrently/) --- Reply to t

Re: [jclouds] JCLOUDS-470. Include region in group name for AWSEC2SecurityGroupExtensi... (#291)

2014-02-14 Thread Andrew Phillips
+1 - looks good to me. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/291#issuecomment-35130627

Re: [jclouds] JCLOUDS-470. Include region in group name for AWSEC2SecurityGroupExtensi... (#291)

2014-02-14 Thread Andrew Phillips
> @@ -41,13 +46,6 @@ > import org.jclouds.net.domain.IpPermission; > import org.jclouds.net.domain.IpProtocol; > > -import com.google.common.base.Function; > -import com.google.common.base.Supplier; > -import com.google.common.cache.LoadingCache; > -import com.google.common.collect.Iterables; >

Re: [jclouds] JCLOUDS-467. Properly iterate over node names for EC2 instance creation. (#292)

2014-02-14 Thread BuildHive
[jclouds » jclouds #844](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/844/) 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/292#issuecomment

Re: [jclouds] JCLOUDS-467. Properly iterate over node names for EC2 instance creation. (#292)

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

Re: [jclouds] JCLOUDS-470. Include region in group name for AWSEC2SecurityGroupExtensi... (#291)

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

Re: [jclouds] JCLOUDS-470. Include region in group name for AWSEC2SecurityGroupExtensi... (#291)

2014-02-14 Thread CloudBees pull request builder plugin
[jclouds-java-7-pull-requests #1073](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/1073/) 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/291#issuecomment-35125312

Re: [jclouds] JCLOUDS-467. Properly iterate over node names for EC2 instance creation. (#292)

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

Re: [jclouds] JCLOUDS-470. Include region in group name for AWSEC2SecurityGroupExtensi... (#291)

2014-02-14 Thread BuildHive
[jclouds » jclouds #843](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/843/) 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/291#issuecomment

Re: [jclouds] Add keystone user add and delete methods. (#290)

2014-02-14 Thread Austin
Thanks! I'll give that a try from here and see if I can get the live tests working. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/290#issuecomment-35120286

[jclouds] JCLOUDS-467. Properly iterate over node names for EC2 instance creation. (#292)

2014-02-14 Thread Andrew Bayer
You can merge this Pull Request by running: git pull https://github.com/abayer/jclouds-1 jclouds-467 Or you can view, comment on it, or merge it online at: https://github.com/jclouds/jclouds/pull/292 -- Commit Summary -- * JCLOUDS-467. Properly iterate over node names for EC2 instance c

[jclouds] JCLOUDS-470. Include region in group name for AWSEC2SecurityGroupExtensi... (#291)

2014-02-14 Thread Andrew Bayer
...on. You can merge this Pull Request by running: git pull https://github.com/abayer/jclouds-1 jclouds-470 Or you can view, comment on it, or merge it online at: https://github.com/jclouds/jclouds/pull/291 -- Commit Summary -- * JCLOUDS-470. Include region in group name for AWSEC2Securi

[jira] [Assigned] (JCLOUDS-470) region missing from group name in AWSEC2SecurityGroupExtension

2014-02-14 Thread Andrew Bayer (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-470?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andrew Bayer reassigned JCLOUDS-470: Assignee: Andrew Bayer > region missing from group name in AWSEC2SecurityGroupExtension >

[jira] [Created] (JCLOUDS-470) region missing from group name in AWSEC2SecurityGroupExtension

2014-02-14 Thread Everett Toews (JIRA)
Everett Toews created JCLOUDS-470: - Summary: region missing from group name in AWSEC2SecurityGroupExtension Key: JCLOUDS-470 URL: https://issues.apache.org/jira/browse/JCLOUDS-470 Project: jclouds

[jira] [Created] (JCLOUDS-469) Add backup support for trove

2014-02-14 Thread Zack Shoylev (JIRA)
Zack Shoylev created JCLOUDS-469: Summary: Add backup support for trove Key: JCLOUDS-469 URL: https://issues.apache.org/jira/browse/JCLOUDS-469 Project: jclouds Issue Type: Improvement

[jira] [Commented] (JCLOUDS-467) nameNodes only taking first name with AWS

2014-02-14 Thread Andrew Bayer (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-467?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13901839#comment-13901839 ] Andrew Bayer commented on JCLOUDS-467: -- I was thinking maybe something like... {cod

[jira] [Commented] (JCLOUDS-423) Implement Rackspace Cloud Files

2014-02-14 Thread Jeremy Daggett (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-423?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13901751#comment-13901751 ] Jeremy Daggett commented on JCLOUDS-423: Running list of all of the PRs that will

Re: [jclouds-labs-openstack] Adds support for Rackspace Cloud Files API (#79)

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

Re: [jclouds-labs-openstack] Adds support for Rackspace Cloud Files API (#79)

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

[jclouds-labs-openstack] Adds support for Rackspace Cloud Files API (#79)

2014-02-14 Thread Jeremy Daggett
This PR adds initial support for the Rackspace Cloud Files API. There are many changes in this PR and will address comments immediately. Thanks. JIRA: https://issues.apache.org/jira/browse/JCLOUDS-423 Additions: - Added support for CloudFilesApi/CDNApi - Added mock/live tests - Refactor

Re: [jclouds] Add keystone user add and delete methods. (#290)

2014-02-14 Thread Everett Toews
> @@ -81,4 +81,21 @@ > */ > Set listRolesOfUserOnTenant(String userId, String > tenantId); > > + /** > +* Creates a new user in keystone. > +* > +* @param userName the name for the new user > +* @param userEmail the email address for the new user > +* @param enabl

Re: [jclouds] Add keystone user add and delete methods. (#290)

2014-02-14 Thread Everett Toews
@trainman419 The live test is failing for me. It's running against Keystone v2 API in Havana. I'm running it with the following command in the `apis/openstack-keystone` dir. mvn clean test -Plive -Dtest.openstack-keystone.endpoint=http://162.242.219.27:5000/v2.0/ -Dtest.openstack-keystone.

[jira] [Commented] (JCLOUDS-468) Nova needs to handle key pair creation better in the ComputeService

2014-02-14 Thread Everett Toews (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-468?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13901544#comment-13901544 ] Everett Toews commented on JCLOUDS-468: --- +1 > Nova needs to handle key pair cre

[jira] [Resolved] (JCLOUDS-425) bad substitution error after compile

2014-02-14 Thread Marc Morata (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-425?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marc Morata resolved JCLOUDS-425. - Resolution: Not A Problem > bad substitution error after compile > -

[jira] [Closed] (JCLOUDS-425) bad substitution error after compile

2014-02-14 Thread Marc Morata (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-425?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marc Morata closed JCLOUDS-425. --- > bad substitution error after compile > > > Key: J

[jira] [Commented] (JCLOUDS-425) bad substitution error after compile

2014-02-14 Thread Marc Morata (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-425?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13901375#comment-13901375 ] Marc Morata commented on JCLOUDS-425: - It worked! Thanks. > bad substitution error

[jira] [Commented] (JCLOUDS-425) bad substitution error after compile

2014-02-14 Thread Ignasi Barrera (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-425?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13901348#comment-13901348 ] Ignasi Barrera commented on JCLOUDS-425: Instead of using the files in "src" you