Re: [jclouds-labs] add docker NetworkAPI (#221)

2015-11-10 Thread Andrea Turli
unfortunately the failure is not related to this PR but to https://jclouds.ci.cloudbees.com/job/jclouds-labs-pull-requests/927/org.apache.jclouds.labs$jdbc/ --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/pull/221#issuecomment-155475255

Re: [jclouds-labs] support docker api v1.18 (#174)

2015-11-10 Thread Andrea Turli
I think #221 supercedes this PR. @jbartece can you please look at it and report here, if #174 can be closed? Thanks! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/pull/174#issuecomment-155463815

Re: [jclouds-labs] add docker NetworkAPI (#221)

2015-11-10 Thread Andrea Turli
Using the following versions ``` $ docker version Client: Version: 1.9.0 API version: 1.21 Go version: go1.4.3 Git commit: 76d6bc9 Built:Tue Nov 3 19:20:09 UTC 2015 OS/Arch: darwin/amd64 Server: Version: 1.9.0 API version: 1.21 Go version: go1.4.3 Git

Re: [jclouds] Fixes tests failing on windows filesystems (#871)

2015-11-10 Thread Andrew Phillips
> @@ -161,14 +164,6 @@ public void testDeleteDirectory() throws IOException { >TestUtils.directoryExists(TARGET_CONTAINER_NAME, false); > } > > - public void testDeleteDirectory_ErrorWhenNotExists() { No longer applicable or..? --- Reply to this email directly or view it on

[jira] [Created] (JCLOUDS-1035) Docker - support Network API

2015-11-10 Thread Andrea Turli (JIRA)
Andrea Turli created JCLOUDS-1035: - Summary: Docker - support Network API Key: JCLOUDS-1035 URL: https://issues.apache.org/jira/browse/JCLOUDS-1035 Project: jclouds Issue Type: New Feature

Re: [jclouds] Fixes tests failing on windows filesystems (#871)

2015-11-10 Thread Zack Shoylev
> + * The ASF licenses this file to You under the Apache License, Version 2.0 > + * (the "License"); you may not use this file except in compliance with > + * the License. You may obtain a copy of the License at > + * > + * http://www.apache.org/licenses/LICENSE-2.0 > + * > + * Unless

Re: [jclouds] Fixes tests failing on windows filesystems (#871)

2015-11-10 Thread Andrew Phillips
> + * The ASF licenses this file to You under the Apache License, Version 2.0 > + * (the "License"); you may not use this file except in compliance with > + * the License. You may obtain a copy of the License at > + * > + * http://www.apache.org/licenses/LICENSE-2.0 > + * > + * Unless

[jira] [Commented] (JCLOUDS-1035) Docker - support Network API

2015-11-10 Thread Andrea Turli (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-1035?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14998798#comment-14998798 ] Andrea Turli commented on JCLOUDS-1035: --- See https://github.com/jclouds/jclouds-labs/pull/221 >

Re: [jclouds] Fixes tests failing on windows filesystems (#871)

2015-11-10 Thread Zack Shoylev
> @@ -161,14 +164,6 @@ public void testDeleteDirectory() throws IOException { >TestUtils.directoryExists(TARGET_CONTAINER_NAME, false); > } > > - public void testDeleteDirectory_ErrorWhenNotExists() { Yes, had to slightly change the spec. --- Reply to this email directly or view

Re: [jclouds-labs] add docker NetworkAPI (#221)

2015-11-10 Thread Ignasi Barrera
> +package org.jclouds.docker.domain; > + > +import static org.jclouds.docker.internal.NullSafeCopies.copyOf; > +import java.util.List; > +import java.util.Map; > + > +import org.jclouds.javax.annotation.Nullable; > +import org.jclouds.json.SerializedNames; > + > +import

Re: [jclouds-labs] add docker NetworkAPI (#221)

2015-11-10 Thread Ignasi Barrera
> + CreateImageOptions options = > CreateImageOptions.Builder.fromImage(ALPINE_IMAGE_TAG); > + InputStream createImageStream = > api.getImageApi().createImage(options); > + consumeStream(createImageStream); > + } > + image =

Re: [jclouds-labs] add docker NetworkAPI (#221)

2015-11-10 Thread Ignasi Barrera
> + container = api.getContainerApi().inspectContainer(container.id()); > + } > + > + @AfterClass > + protected void tearDown() { > + if (container != null) { > + api.getContainerApi().stopContainer(container.id()); > +

Re: [jclouds-labs] add docker NetworkAPI (#221)

2015-11-10 Thread Ignasi Barrera
Nice job updating the model to the latest version @andreaturli. Thanks! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/pull/221#issuecomment-155574738

Re: [jclouds] Fixes tests failing on windows filesystems (#871)

2015-11-10 Thread Zack Shoylev
@demobox @andrewgaul Updated again. Should be good to merge? Hopefully I didn't miss anything, it's a bit larger in scope than originally planned. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/871#issuecomment-155569257

Re: [jclouds-labs] add docker NetworkAPI (#221)

2015-11-10 Thread Ignasi Barrera
> + public void testGetNetwork() { > + network = api().inspectNetwork(network.id()); > + assertNotNull(network); > + } > + > + @Test(dependsOnMethods = "testGetNetwork") > + public void testAttachContainerToNetwork() { > + api().connectContainerToNetwork(network.id(),

Re: [jclouds-site] Add foundation links (#175)

2015-11-10 Thread Ignasi Barrera
> @@ -52,6 +52,16 @@ > href="https://cwiki.apache.org/confluence/display/JCLOUDS/How+to+Contribute+Documentation;>How > To Contribute Documentation > > > + > + data-toggle="dropdown">About Apache

Re: [jclouds-labs] add docker NetworkAPI (#221)

2015-11-10 Thread Ignasi Barrera
> + @Named("networks:list") > + @GET > + @Fallback(EmptyListOnNotFoundOr404.class) > + List listNetworks(); > + > + /** > +* @param network the network’s configuration (@see BindToJsonPayload) > +* @return a new network > +*/ > + @Named("network:create") > + @POST > +

Re: [jclouds-labs] add docker NetworkAPI (#221)

2015-11-10 Thread Ignasi Barrera
>} > >public Builder fromNetworkSettings(NetworkSettings in) { > return > this.ipAddress(in.ipAddress()).ipPrefixLen(in.ipPrefixLen()).gateway(in.gateway()).bridge(in.bridge()) > - .portMapping(in.portMapping()).ports(in.ports()); > + >

Re: [jclouds-labs] add docker NetworkAPI (#221)

2015-11-10 Thread Ignasi Barrera
> + } > + > + @Test(dependsOnMethods = "testGetNetwork") > + public void testAttachContainerToNetwork() { > + api().connectContainerToNetwork(network.id(), container.id()); > + Container foundContainer = > api.getContainerApi().inspectContainer(container.id()); > + >

Re: [jclouds-labs] add docker NetworkAPI (#221)

2015-11-10 Thread Ignasi Barrera
> + > + try { > + api.removeNetwork(networkId); > + assertSent(server, "DELETE", "/networks/" + networkId); > + > + } finally { > + server.shutdown(); > + } > + } > + > + public void testConnectContainerToNetwork() throws Exception { > +

[jira] [Updated] (JCLOUDS-1034) AWS SQS - MessageAttributes not currently supported

2015-11-10 Thread Andrew Gaul (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-1034?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andrew Gaul updated JCLOUDS-1034: - Issue Type: New Feature (was: Improvement) > AWS SQS - MessageAttributes not currently

[jira] [Commented] (JCLOUDS-1034) AWS SQS - MessageAttributes not currently supported

2015-11-10 Thread Andrew Gaul (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-1034?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14999444#comment-14999444 ] Andrew Gaul commented on JCLOUDS-1034: -- Odd, {{org.jclouds.sqs.features.MessageApi.receive}}

[jira] [Updated] (JCLOUDS-1034) AWS SQS - MessageAttributes not currently supported

2015-11-10 Thread Andrew Gaul (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-1034?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andrew Gaul updated JCLOUDS-1034: - Description: After a jclouds forum inquiry it appears that "MessageAttributes" are not

Re: [jclouds] Fixes tests failing on windows filesystems (#871)

2015-11-10 Thread Andrew Phillips
> Should be good to merge? Looks good to me, @zack-shoylev - +1. Thanks for the updates! @andrewgaul may have some comments on regarding Windows as an eventually consistent store..? --- Reply to this email directly or view it on GitHub:

Re: [jclouds] Remove HPCloud from the aggregator poms (#877)

2015-11-10 Thread Ignasi Barrera
> @@ -44,11 +44,6 @@ >${project.version} > > > - org.apache.jclouds.api > - chef > - ${project.version} > - > - This one was duplicated. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/877/files#r44386232

[jclouds] Remove HPCloud from the aggregator poms (#877)

2015-11-10 Thread Ignasi Barrera
You can view, comment on, or merge this pull request online at: https://github.com/jclouds/jclouds/pull/877 -- Commit Summary -- * Remove HPCloud from the aggregator poms -- File Changes -- M all/pom.xml (5) M allblobstore/pom.xml (5) M allcompute/pom.xml (5) -- Patch Links

[jira] [Created] (JCLOUDS-1034) AWS SQS - MessagAttributes not currently supprted

2015-11-10 Thread Massimo Mirabito (JIRA)
Massimo Mirabito created JCLOUDS-1034: - Summary: AWS SQS - MessagAttributes not currently supprted Key: JCLOUDS-1034 URL: https://issues.apache.org/jira/browse/JCLOUDS-1034 Project: jclouds

Re: [jclouds-site] add softlayer getting started (#174)

2015-11-10 Thread Andrea Turli
Thanks @nacx ! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-site/pull/174#issuecomment-155386385

Re: [jclouds] Fixes tests failing on windows filesystems (#871)

2015-11-10 Thread Andrew Phillips
> + * The ASF licenses this file to You under the Apache License, Version 2.0 > + * (the "License"); you may not use this file except in compliance with > + * the License. You may obtain a copy of the License at > + * > + * http://www.apache.org/licenses/LICENSE-2.0 > + * > + * Unless

Re: [jclouds-site] Add foundation links (#175)

2015-11-10 Thread Ignasi Barrera
> @@ -52,6 +52,16 @@ > href="https://cwiki.apache.org/confluence/display/JCLOUDS/How+to+Contribute+Documentation;>How > To Contribute Documentation > > > + > + data-toggle="dropdown">About Apache

Re: [jclouds-site] Add foundation links (#175)

2015-11-10 Thread Andrew Phillips
> @@ -52,6 +52,16 @@ > href="https://cwiki.apache.org/confluence/display/JCLOUDS/How+to+Contribute+Documentation;>How > To Contribute Documentation > > > + > + data-toggle="dropdown">About Apache

Re: [jclouds-site] Add foundation links (#175)

2015-11-10 Thread Andrew Phillips
> @@ -52,6 +52,16 @@ > href="https://cwiki.apache.org/confluence/display/JCLOUDS/How+to+Contribute+Documentation;>How > To Contribute Documentation > > > + > + data-toggle="dropdown">About Apache

Re: [jclouds] Fixes tests failing on windows filesystems (#871)

2015-11-10 Thread Andrew Phillips
> @@ -477,8 +478,10 @@ public String putBlob(final String containerName, final > Blob blob) throws IOExce > return base16().lowerCase().encode(actualHashCode.asBytes()); >} catch (IOException ex) { > if (outputFile != null) { > -if (!outputFile.delete()) {

Re: [jclouds] Fixes tests failing on windows filesystems (#871)

2015-11-10 Thread Andrew Phillips
> } >} > > + try { > + delete(fileToBeDeleted); > + } catch (IOException e) { > + logger.debug("%s", e); Same comment about the log message --- Reply to this email directly or view it on GitHub:

Re: [jclouds] Fixes tests failing on windows filesystems (#871)

2015-11-10 Thread Andrew Phillips
> @@ -477,8 +478,10 @@ public String putBlob(final String containerName, final > Blob blob) throws IOExce > return base16().lowerCase().encode(actualHashCode.asBytes()); >} catch (IOException ex) { > if (outputFile != null) { > -if (!outputFile.delete()) {

Re: [jclouds] Fixes tests failing on windows filesystems (#871)

2015-11-10 Thread Andrew Phillips
> @@ -39,7 +41,12 @@ > protected void configure() { >bind(BlobStore.class).to(LocalBlobStore.class); >install(new BlobStoreObjectModule()); > - bind(ConsistencyModel.class).toInstance(ConsistencyModel.STRICT); > + if (isWindows()) { > +

Re: [jclouds] Fixes tests failing on windows filesystems (#871)

2015-11-10 Thread Andrew Phillips
> + for (int n = 0; n < 10; n++) { > + try { > +Files.delete(file.toPath()); > + } catch (DirectoryNotEmptyException dnee) { > +// A previous file delete operation did not finish before this > call > +

[jira] [Updated] (JCLOUDS-1034) AWS SQS - MessageAttributes not currently supported

2015-11-10 Thread Andrew Gaul (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-1034?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andrew Gaul updated JCLOUDS-1034: - Priority: Major (was: Blocker) > AWS SQS - MessageAttributes not currently supported >

[jira] [Updated] (JCLOUDS-1034) AWS SQS - MessageAttributes not currently supported

2015-11-10 Thread Andrew Gaul (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-1034?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andrew Gaul updated JCLOUDS-1034: - Summary: AWS SQS - MessageAttributes not currently supported (was: AWS SQS - MessagAttributes