Jenkins build is back to stable : jclouds #2735

2014-08-28 Thread jenkins-no-reply
See

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

2014-08-28 Thread jenkins-no-reply
See

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

2014-08-28 Thread BuildHive
See

Re: [jclouds] Set Content-MD5 only if ETag has correct format (#493)

2014-08-28 Thread Andrew Gaul
@demobox Oops I missed your comment about where we test this. Honestly I need to think about how we want to represent multi-part ETag and I am not sure we need to sanity check bad ETag, similar to how we don't check malformed integers in response parsing. --- Reply to this email directly or vi

Re: [jclouds-labs-openstack] Adds support for the Security Group extension to neutron (#132)

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

Re: [jclouds] Set Content-MD5 only if ETag has correct format (#493)

2014-08-28 Thread Andrew Gaul
Closed #493. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/493#event-158810157

Re: [jclouds] Set Content-MD5 only if ETag has correct format (#493)

2014-08-28 Thread Andrew Gaul
Pushed to master as df996f3231005b1ef294223d20efbabe74a23de7 and 1.8.x as f2d897d9774c2c0225c199c7f2f46971637327d6. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/493#issuecomment-53837257

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Zack Shoylev
> +Objects.equal(this.tenantId, that.tenantId) && > +Objects.equal(this.floatingNetworkId, that.floatingNetworkId) && > +Objects.equal(this.fixedIpAddress, that.fixedIpAddress) && > +Objects.equal(this.floatingIpAddress, that.floatingIpAddress) && > +

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Jeremy Daggett
> +Objects.equal(this.tenantId, that.tenantId) && > +Objects.equal(this.floatingNetworkId, that.floatingNetworkId) && > +Objects.equal(this.fixedIpAddress, that.fixedIpAddress) && > +Objects.equal(this.floatingIpAddress, that.floatingIpAddress) && > +

Jenkins build became unstable: jclouds » jclouds #1566

2014-08-28 Thread BuildHive
See

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

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

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

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

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Zack Shoylev
> + @Path("/{id}") > + @SelectJson("floatingip") > + @Fallback(NullOnNotFoundOr404.class) > + @Nullable > + FloatingIP get(@PathParam("id") String id); > + > + /** > +* This operation creates a floating IP. > +* > +* @param floatingIP Options for creating a floatingip > +

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Andrew Phillips
> + server.shutdown(); > + } > + } > + > + public void testListPagedFloatingIP() throws IOException, > InterruptedException, URISyntaxException { > + MockWebServer server = mockOpenStackServer(); > + server.enqueue(addCommonHeaders(new > MockResponse().setBody(stringFro

[jira] [Created] (JCLOUDS-689) Be able to name firewall rules

2014-08-28 Thread Sunil Shah (JIRA)
Sunil Shah created JCLOUDS-689: -- Summary: Be able to name firewall rules Key: JCLOUDS-689 URL: https://issues.apache.org/jira/browse/JCLOUDS-689 Project: jclouds Issue Type: Improvement

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Andrew Phillips
> + assertAuthentication(server); > + assertRequest(server.takeRequest(), "GET", > "/v2.0/floatingips?limit=2&marker=abcdefg"); > + > + /* > + * Check response > + */ > + assertNotNull(floatingIPs); > + assertEquals(floatingIPs.size(), 2);

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Andrew Phillips
> + @Path("/{id}") > + @SelectJson("floatingip") > + @Fallback(NullOnNotFoundOr404.class) > + @Nullable > + FloatingIP get(@PathParam("id") String id); > + > + /** > +* This operation creates a floating IP. > +* > +* @param floatingIP Options for creating a floatingip > +

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Zack Shoylev
> + } > + > + private abstract static class Builder { > + protected FloatingIP floatingIP; > + > + /** > + * No-parameters constructor used when updating. > + * */ > + private Builder() { > + floatingIP = new FloatingIP(); > + } > + > + protected abs

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Andrew Phillips
> + public static class CreateOptions extends FloatingIP { > + /** > + * Copy constructor > + */ > + private CreateOptions(FloatingIP floatingIP) { > + super(floatingIP); > + } > + } > + public static class UpdateOptions extends FloatingIP { > + /** > +

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Zack Shoylev
> + @Named("tenant_id") > + private String tenantId; > + // Only mandatory attribute when creating > + @Named("floating_network_id") > + private String floatingNetworkId; > + @Named("fixed_ip_address") > + private String fixedIpAddress; > + @Named("floating_ip_address") > + privat

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Andrew Phillips
> +network = networkApi.create( > + > Network.createOptions("jclouds-network-test").external(true).networkType(NetworkType.LOCAL).build()); > +assertNotNull(network); > + > +ipv4SubnetId = > subnetApi.create(Subnet.createOptions(network.getId()

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Andrew Phillips
> + */ > + assertNotNull(floatingIP); > + assertEquals(floatingIP.getRouterId(), > "d23abc8d-2991-4a55-ba98-2aaea84cc72f"); > + assertEquals(floatingIP.getTenantId(), > "4969c491a3c74ee4af974e6d800c62de"); > + assertEquals(floatingIP.getFloatingNetworkId()

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Zack Shoylev
> + "floating_ip_address", "port_id"}) > + private FloatingIP(String id, String routerId, String tenantId, String > floatingNetworkId, String fixedIpAddress, > + String floatingIpAddress, String portId) { > + this.id = id; > + this.routerId = routerId; > + this.ten

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Zack Shoylev
> + public static CreateBuilder createOptions(String floatingNetworkId) { > + return new CreateBuilder(floatingNetworkId); > + } > + > + /** > +* @return the Builder for updating a floating IP > +*/ > + public static UpdateBuilder updateOptions() { > + return new UpdateBui

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Zack Shoylev
> + } > + > + public static class UpdateBuilder extends Builder { > + /** > + * Supply required properties for updating a Builder > + */ > + private UpdateBuilder() { > + } > + > + /** > + * Provide the floating ip address for this Floating IP. > + *

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Zack Shoylev
> + public static class CreateOptions extends FloatingIP { > + /** > + * Copy constructor > + */ > + private CreateOptions(FloatingIP floatingIP) { > + super(floatingIP); > + } > + } > + public static class UpdateOptions extends FloatingIP { > + /** > +

Re: [jclouds-labs] Backport artifactId typo fix (#66)

2014-08-28 Thread Andrew Gaul
Abandoning since we released 1.8.0. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/pull/66#issuecomment-53821532

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Zack Shoylev
> + */ > +package org.jclouds.openstack.neutron.v2.domain; > + > +import com.google.common.collect.ImmutableSet; > +import org.jclouds.openstack.v2_0.domain.Link; > +import org.jclouds.openstack.v2_0.domain.PaginatedCollection; > + > +import java.beans.ConstructorProperties; > + > +/** > + * A coll

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Zack Shoylev
> +network = networkApi.create( > + > Network.createOptions("jclouds-network-test").external(true).networkType(NetworkType.LOCAL).build()); > +assertNotNull(network); > + > +ipv4SubnetId = > subnetApi.create(Subnet.createOptions(network.getId()

Re: [jclouds-labs-openstack] Adds support for the Security Group extension to neutron (#132)

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

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Zack Shoylev
> + > floatingIPApi.create(FloatingIP.createOptions(network.getId()).build()); > + > + /* List and Get test */ > +Set floatingIPs = > floatingIPApi.list().concat().toSet(); > +FloatingIP floatingIPList = floatingIPs.iterator().next(); > +floa

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Zack Shoylev
> + */ > + assertNotNull(floatingIP); > + assertEquals(floatingIP.getRouterId(), > "d23abc8d-2991-4a55-ba98-2aaea84cc72f"); > + assertEquals(floatingIP.getTenantId(), > "4969c491a3c74ee4af974e6d800c62de"); > + assertEquals(floatingIP.getFloatingNetworkId()

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

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

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Zack Shoylev
> + assertAuthentication(server); > + assertRequest(server.takeRequest(), "GET", > "/v2.0/floatingips?limit=2&marker=abcdefg"); > + > + /* > + * Check response > + */ > + assertNotNull(floatingIPs); > + assertEquals(floatingIPs.size(), 2);

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

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

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Zack Shoylev
> + server.shutdown(); > + } > + } > + > + public void testListPagedFloatingIP() throws IOException, > InterruptedException, URISyntaxException { > + MockWebServer server = mockOpenStackServer(); > + server.enqueue(addCommonHeaders(new > MockResponse().setBody(stringFro

Jenkins build is unstable: jclouds #2734

2014-08-28 Thread jenkins-no-reply
See

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

2014-08-28 Thread jenkins-no-reply
See

Re: [jclouds-labs] Add vSphere support (#61)

2014-08-28 Thread Ignasi Barrera
Hi @igreenfield, I've started [this thread](http://markmail.org/message/zjkxxozjilwycqgg) thread in the [dev@ mailing list](http://jclouds.apache.org/community/) to discuss the need to remove the vijava dependency. I encourage you to subscribe and participate in the discussion! --- Reply to

Re: [jclouds-labs-google] JCLOUDS-458:Multipart Related Upload (#46)

2014-08-28 Thread Andrew Gaul
> +import com.google.gson.Gson; > +import static com.google.common.base.Preconditions.checkNotNull; > + > +public class MultipartUploadBinder implements MapBinder { > + > + private final String BOUNDARY_HEADER = "multipart_boundary"; > + > + @Override > + public R bindToRequest(R request, Ma

Re: [jclouds-labs-google] JCLOUDS-458:Multipart Related Upload (#46)

2014-08-28 Thread Andrew Gaul
> + hcCrc32c = hf2.newHasher().putBytes(byteSource.read()).hash(); > + > + > template.contentType("image/jpeg").addAcl(oacl).size(contentLength).name(MULTIPART_UPLOAD_OBJECT) > + > .contentLanguage("en").contentDisposition("attachment").md5Hash(hcMd5) > + .cu

Re: [jclouds-labs-google] JCLOUDS-458:Multipart Related Upload (#46)

2014-08-28 Thread Andrew Gaul
> + ByteSource byteSource = TestUtils.randomByteSource().slice(0, > contentLength); > + ByteSourcePayload payload = Payloads.newByteSourcePayload(byteSource); > + PayloadEnclosingImpl payloadImpl = new PayloadEnclosingImpl(payload); > + > + ObjectTemplate template = new ObjectT

Re: [jclouds-labs-google] JCLOUDS-458:Multipart Related Upload (#46)

2014-08-28 Thread Andrew Gaul
> + String contentType = checkNotNull(template.getContentType(), > "contentType"); > + Long lenght = checkNotNull(template.getSize(), "contetLength"); > + > + StringPayload jsonPayload = Payloads.newStringPayload(new > Gson().toJson(template)); > + > + payload.getContentMetada

Re: [jclouds-labs-google] JCLOUDS-458:Multipart Related Upload (#46)

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

Re: [jclouds-labs-google] JCLOUDS-458:Multipart Related Upload (#46)

2014-08-28 Thread BuildHive
[jclouds » jclouds-labs-google #1301](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-labs-google/1301/) SUCCESS This pull request looks good [(what's this?)](https://www.cloudbees.com/what-is-buildhive) --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jc

[jclouds-labs-google] JCLOUDS-458:Multipart Related Upload (#46)

2014-08-28 Thread Bhathiya
"If you have metadata that you want to send along with the data to upload, you can make a single multipart/related request. As with simple, media-only requests, this is a good choice if the data you are sending is small enough to upload again in its entirety if the connection fails." You can mer

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Zack Shoylev
> + @POST > + @SelectJson("floatingip") > + FloatingIP create(@WrapWith("floatingip") FloatingIP.CreateOptions > floatingIP); > + > + /** > +* Update a floatingip > +* > +* @param id the id of the floatingip to update > +* @param floatingIP Contains only the attributes to u

[jira] [Commented] (JCLOUDS-575) Support multiple ranges in getBlob

2014-08-28 Thread Michael Pavlovsky (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-575?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14114264#comment-14114264 ] Michael Pavlovsky commented on JCLOUDS-575: --- I had a discussion on openstack sw

Re: [jclouds] JCLOUDS-686: Remove changesSince() query from base PaginationOptions (#499)

2014-08-28 Thread Jeremy Daggett
> @@ -44,7 +42,11 @@ public PaginationOptions queryParameters(Multimap String> queryParams) { > > /** > * Only return objects changed since this time. > +* > +* @deprecated The {@code changes-since} query does not apply to all > OpenStack APIs. @everett-toews Since this is bein

Re: [jclouds] JCLOUDS-686: Remove changesSince() query from base PaginationOptions (#499)

2014-08-28 Thread Everett Toews
> @@ -44,7 +42,11 @@ public PaginationOptions queryParameters(Multimap String> queryParams) { > > /** > * Only return objects changed since this time. > +* > +* @deprecated The {@code changes-since} query does not apply to all > OpenStack APIs. The Javadoc needs to link to what

[jira] [Commented] (JCLOUDS-575) Support multiple ranges in getBlob

2014-08-28 Thread Daniel Hsueh (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-575?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14114198#comment-14114198 ] Daniel Hsueh commented on JCLOUDS-575: -- My guess is the aws-s3 response is compliant

[jira] [Commented] (JCLOUDS-575) Support multiple ranges in getBlob

2014-08-28 Thread Daniel Hsueh (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-575?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14114190#comment-14114190 ] Daniel Hsueh commented on JCLOUDS-575: -- I recently learned of http://tools.ietf.org/

Re: [jclouds] JCLOUDS-686: Remove changesSince() query from base PaginationOptions (#499)

2014-08-28 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #1124](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1124/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/499#issuecomment-53769469

Re: [jclouds-labs-google] JCLOUDS-458: Basic Object operations with live tests (#45)

2014-08-28 Thread Andrew Gaul
Closed #45. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-google/pull/45#event-158580598

Re: [jclouds-labs-google] JCLOUDS-458: Basic Object operations with live tests (#45)

2014-08-28 Thread Andrew Gaul
Tested locally, converted some `toStringAndClose` to `toByteArrayAndClose` methods, removed trailing whitespace, changed DOS to Unix linux endings, reworded commit message, and pushed to master as 155fc11246705cdce1658beb20182a44d01f080e. --- Reply to this email directly or view it on GitHub: h

[jira] [Created] (JCLOUDS-688) Nova ServerApi.createImageFromServer() does not consider metadata

2014-08-28 Thread Jeremy Daggett (JIRA)
Jeremy Daggett created JCLOUDS-688: -- Summary: Nova ServerApi.createImageFromServer() does not consider metadata Key: JCLOUDS-688 URL: https://issues.apache.org/jira/browse/JCLOUDS-688 Project: jcloud

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

2014-08-28 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=14114077#comment-14114077 ] ASF subversion and git services commented on JCLOUDS-458: - Commit

Re: [jclouds] JCLOUDS-686: Remove changesSince() query from base PaginationOptions (#499)

2014-08-28 Thread BuildHive
[jclouds » jclouds #1565](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1565/) 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/499#issuecomme

Re: [jclouds-labs-google] JCLOUDS-458: Basic Object operations with live tests (#45)

2014-08-28 Thread Andrew Gaul
> + } > + > + public StorageClass getStorageClass() { > + return storageClass; > + } > + > + public Long getSize() { > + return size; > + } > + > + private String getMd5Hash() { > + return md5Hash; > + } > + > + public HashCode getMd5HashCode() { > + HashCode hc

Re: [jclouds] JCLOUDS-686: Remove changesSince() query from base PaginationOptions (#499)

2014-08-28 Thread CloudBees pull request builder plugin
[jclouds-pull-requests-java-6 #35](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests-java-6/35/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/499#issuecomment-53757862

[jira] [Commented] (JCLOUDS-686) Remove OpenStack PaginationOptions.changesSince()

2014-08-28 Thread Jeremy Daggett (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-686?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14113975#comment-14113975 ] Jeremy Daggett commented on JCLOUDS-686: https://github.com/jclouds/jclouds-labs-

[jclouds] JCLOUDS-686: Remove changesSince() query from base PaginationOptions (#499)

2014-08-28 Thread Jeremy Daggett
This PR deprecates the `changesSince(Date ifModifiedSince)` method in PaginationOptions. `changes-since` is specific to Nova (and [Glance](https://github.com/jclouds/jclouds-labs-openstack/pull/137)) and has moved to the Nova `ListOptions` class. Nova `ListOptions.maxResults(int maxKeys)` has

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

2014-08-28 Thread BuildHive
See

[jira] [Created] (JCLOUDS-687) Configurable API/Provider Default Page Size

2014-08-28 Thread Jeremy Daggett (JIRA)
Jeremy Daggett created JCLOUDS-687: -- Summary: Configurable API/Provider Default Page Size Key: JCLOUDS-687 URL: https://issues.apache.org/jira/browse/JCLOUDS-687 Project: jclouds Issue Type:

Re: [jclouds] JCLOUDS-514: Add ability to attach block volumes at boot through the Nova ServerApi (#326)

2014-08-28 Thread Jeremy Daggett
> @@ -98,6 +98,51 @@ public String toString() { > > } > > + /** > +* A representation of block device that should be attached to the > instance to be launched > +* > +*/ > + public static class BlockDeviceMapping { I would extract this class into its own domain class `org

Re: [jclouds] jclouds Development Tools (#459)

2014-08-28 Thread Everett Toews
Nope! Merged. Thanks @demobox --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/459#issuecomment-53735693

Re: [jclouds] jclouds Development Tools (#459)

2014-08-28 Thread Everett Toews
Closed #459. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/459#event-158461684

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Andrew Phillips
> + @POST > + @SelectJson("floatingip") > + FloatingIP create(@WrapWith("floatingip") FloatingIP.CreateOptions > floatingIP); > + > + /** > +* Update a floatingip > +* > +* @param id the id of the floatingip to update > +* @param floatingIP Contains only the attributes to u

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Ignasi Barrera
> + @POST > + @SelectJson("floatingip") > + FloatingIP create(@WrapWith("floatingip") FloatingIP.CreateOptions > floatingIP); > + > + /** > +* Update a floatingip > +* > +* @param id the id of the floatingip to update > +* @param floatingIP Contains only the attributes to u

Re: [jclouds] Ignore jEnv files (#495)

2014-08-28 Thread Andrew Phillips
> I've just opened RAT-171 and attached a patch. Voted for the issue and added a comment ;-) Thanks! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/495#issuecomment-53712664

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Andrew Phillips
> + @POST > + @SelectJson("floatingip") > + FloatingIP create(@WrapWith("floatingip") FloatingIP.CreateOptions > floatingIP); > + > + /** > +* Update a floatingip > +* > +* @param id the id of the floatingip to update > +* @param floatingIP Contains only the attributes to u

Re: [jclouds] Fix Karaf bundle scoping in Cloud Load Balancers (#490)

2014-08-28 Thread Andrew Phillips
> Yes, it fixed the jclouds-karaf dependency resolution. Nice! Thanks for confirming. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/490#issuecomment-53711590

Re: [jclouds] Move jclouds-chef to the main jclouds repo (#458)

2014-08-28 Thread Andrew Phillips
> I want to do it right this time :) *grin*. Good luck! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/458#issuecomment-53711524

Re: [jclouds] Add top-level legacy directory (#489)

2014-08-28 Thread Andrew Phillips
> It's always better to discuss and propose things +1. Thanks for spurring on the discussion, @jdaggett! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/489#issuecomment-53711497

Re: [jclouds-labs-google] JCLOUDS-458: Basic Object operations with live tests (#45)

2014-08-28 Thread Bhathiya
> + } > + > + public StorageClass getStorageClass() { > + return storageClass; > + } > + > + public Long getSize() { > + return size; > + } > + > + private String getMd5Hash() { > + return md5Hash; > + } > + > + public HashCode getMd5HashCode() { > + HashCode hc

Re: [jclouds-labs-google] JCLOUDS-458: Basic Object operations with live tests (#45)

2014-08-28 Thread Bhathiya
Chnages 1- Moved reverse(byte []) to DoamainUtils 2- Changed String to Long ContentLenght in ObjectApi 3-Test Payload Hashcode tests SimpleUpload only uploads the Media part and md5 or Crc32c can not be sent with request. What we can only do is client side validation .I have added More MD5/C

Re: [jclouds-labs-google] JCLOUDS-458: Basic Object operations with live tests (#45)

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

Re: [jclouds-labs-google] JCLOUDS-458: Basic Object operations with live tests (#45)

2014-08-28 Thread BuildHive
[jclouds » jclouds-labs-google #1297](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-labs-google/1297/) SUCCESS This pull request looks good [(what's this?)](https://www.cloudbees.com/what-is-buildhive) --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jc

Jenkins build is back to stable : jclouds » jclouds-labs-google #1297

2014-08-28 Thread BuildHive
See

Re: [jclouds] Ignore jEnv files (#495)

2014-08-28 Thread Ignasi Barrera
Absolutely :) I've just opened [RAT-171](https://issues.apache.org/jira/browse/RAT-171) and attached a patch. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/495#issuecomment-53694862

Re: [jclouds] Move jclouds-chef to the main jclouds repo (#458)

2014-08-28 Thread Ignasi Barrera
Yes, i have it in mind, but I want to have all working locally before reopening it, to avoid the karaf issues that appeared after merging it. I want to do it right this time :) --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/458#issuecomment-5368764

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Ignasi Barrera
> + @POST > + @SelectJson("floatingip") > + FloatingIP create(@WrapWith("floatingip") FloatingIP.CreateOptions > floatingIP); > + > + /** > +* Update a floatingip > +* > +* @param id the id of the floatingip to update > +* @param floatingIP Contains only the attributes to u

Re: [jclouds-labs-openstack] Adds support for Floating IP (#135)

2014-08-28 Thread Ignasi Barrera
> + @POST > + @SelectJson("floatingip") > + FloatingIP create(@WrapWith("floatingip") FloatingIP.CreateOptions > floatingIP); > + > + /** > +* Update a floatingip > +* > +* @param id the id of the floatingip to update > +* @param floatingIP Contains only the attributes to u