Re: [jclouds-labs] Cloudsigma2 subscription calc fix (#41)

2013-12-09 Thread gcbirzan
Sorry for the late response, some urgent stuff came up. There is an error, albeit misleading, when doing that, it says you do not have TAG permission on that resource. I've also tried this several times, looked at our acceptance tests, they all do the same and it works. The only thing I can do

Re: [jclouds] Map BlobBuilder.expires to Swift X-Delete-At (#227)

2013-12-09 Thread CloudBees pull request builder plugin
[jclouds-java-7-pull-requests #914](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/914/) 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/227#issuecomment-30204939

Re: [jclouds-labs-openstack] Replacing strange chars in pom.xml with spaces (#53)

2013-12-09 Thread Ignasi Barrera
Then you have mine :) +1 --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-openstack/pull/53#issuecomment-30204871

Re: [jclouds] Map BlobBuilder.expires to Swift X-Delete-At (#227)

2013-12-09 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #452](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/452/) 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/227#issuecomment-30204816

Re: [jclouds] Map BlobBuilder.expires to Swift X-Delete-At (#227)

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

Jenkins build became unstable: jclouds » jclouds #682

2013-12-09 Thread BuildHive
See

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

2013-12-09 Thread BuildHive
See

[jclouds] Map BlobBuilder.expires to Swift X-Delete-At (#227)

2013-12-09 Thread Andrew Gaul
Originally reported at: http://stackoverflow.com/questions/20485579/expiring-swift-objects-with-jclouds You can merge this Pull Request by running: git pull https://github.com/maginatics/jclouds swift-expires Or you can view, comment on it, or merge it online at: https://github.com/jclouds/

Re: [jclouds] Allow the HeaderParam annotation to be used in a Caller. (#226)

2013-12-09 Thread Andrew Phillips
>Multimap queryParams = addQueryParams(tokenValues, > invocation); > - Multimap headers = buildHeaders(tokenValues, > invocation); > + > + Multimap headers; > + > + if (caller != null) { > + headers = buildHeaders(tokenValues, caller); > + headers.putAll(bui

Re: [jclouds-karaf] Allow stdin local file for streaming writes (#23)

2013-12-09 Thread Andrew Gaul
I have been mucking with the Checkstyle config, although jclouds-karaf has a different parent in pom.xml than other repositories. @iocanel Any suggestions on how best to resolve this? --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-karaf/pull/23#issuecom

Re: Not able to bootstrap a node with chef server 10

2013-12-09 Thread Jayant Kaushal
I think that seems to be the issue. I am creating node in AWS. and my chef server is in private VPN. Dont think node can access chef server in my VPN without public IP. thanks Jayant Kaushal +91-9060519860 On Mon, Dec 9, 2013 at 7:41 PM, Jayant Kaushal wrote: > Ok. yes got it. > but another pr

Re: [jclouds] Allow the HeaderParam annotation to be used in a Caller. (#226)

2013-12-09 Thread Andrew Phillips
>Multimap queryParams = addQueryParams(tokenValues, > invocation); > - Multimap headers = buildHeaders(tokenValues, > invocation); > + > + Multimap headers; > + > + if (caller != null) { > + headers = buildHeaders(tokenValues, caller); Just to ensure we're backward

Re: [jclouds-karaf] Allow stdin local file for streaming writes (#23)

2013-12-09 Thread Andrew Phillips
> Can this be merged? I think we first need to address a bunch of new [Checkstyle violations](https://jclouds.ci.cloudbees.com/job/jclouds-karaf-pull-requests/18/org.apache.jclouds.karaf$cache/violations/)? --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds

Re: [jclouds] Allow Guava MediaType for Content-Type (#44)

2013-12-09 Thread Andrew Phillips
@andrewgaul: one small comment left about potentially marking the `contentType` args as `@Nullable`? Otherwise, JIRA issue is [JCLOUDS-402](https://issues.apache.org/jira/browse/JCLOUDS-402). --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/44#issuec

Re: [jclouds] Allow Guava MediaType for Content-Type (#44)

2013-12-09 Thread Andrew Phillips
> @@ -107,6 +108,8 @@ > >PayloadBlobBuilder contentMD5(byte[] md5); > > + PayloadBlobBuilder contentType(MediaType contentType); Mark `MediaType` (and the `String` version below, actually) as `@Nullable`? --- Reply to this email directly or view it on GitHub: https://github.com/j

Re: [jclouds-labs-openstack] Replacing strange chars in pom.xml with spaces (#53)

2013-12-09 Thread Andrew Phillips
@nacx: _I_ certainly think so! Was waiting for someone else to have a look and +1... --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-openstack/pull/53#issuecomment-30201486

Re: [jclouds] Make BlobName Function type-safe (#38)

2013-12-09 Thread Andrew Phillips
See [JCLOUDS-401](https://issues.apache.org/jira/browse/JCLOUDS-401). @andrewgaul: could you amend the commit message to reference the JIRA issue and rebase? --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/38#issuecomment-30201315

Re: [jclouds] Allowing Guava MediaType for Content-Type (#225)

2013-12-09 Thread Andrew Phillips
The Clojure ugliness looks pretty significant, at least to me. And if one variant of `contentType` is nullable, why not the other one, I guess. Closing this one out - let's stick with #44... --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/225#issuec

Re: [jclouds] Allow Guava MediaType for Content-Type (#44)

2013-12-09 Thread Andrew Phillips
Seeing what is required in Clojure to allow `MediaType` to be non-nullable, I'd say we stick with this one, after all ;-) --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/44#issuecomment-30200986

Re: [jclouds] JCLOUDS-356 multipart uploads with InputStream payloads (#192)

2013-12-09 Thread Andrew Phillips
Thanks for the explanations, @eevans! Perhaps add a comment in the test to explain why `InputStream` is used rather than `File`? Otherwise, fine to leave as-is. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/192#issuecomment-30200199

Re: [jclouds] Adds some easy-of-use for openstack mock tests (#224)

2013-12-09 Thread Zack Shoylev
> +* The name of the resource. > +* @return The content of the resource > +*/ > + public String stringFromResource(String resourceName) { > + try { > + return > Strings2.toStringAndClose(getClass().getResourceAsStream(resourceName)); > + } catch (IOExcepti

Re: [jclouds] Adds some easy-of-use for openstack mock tests (#224)

2013-12-09 Thread Zack Shoylev
> @@ -96,4 +112,90 @@ public void enqueueResponse(MockResponse response) { > >return dispatcher; > } > + > + /** > +* Ensure json headers are included > +* */ > + public MockResponse addJsonHeaders(MockResponse mockResponse) { Good point in names. --- Reply to this email

Re: [jclouds] Adds some easy-of-use for openstack mock tests (#224)

2013-12-09 Thread Zack Shoylev
> */ > public class BaseOpenStackMockTest { > > - public static final String accessRackspace = > "{\"access\":{\"token\":{\"id\":\"b84f4a37-5126-4603-9521-ccd0665fbde1\",\"expires\":\"2013-04-13T16:49:57.000-05:00\",\"tenant\":{\"id\":\"123123\",\"name\":\"123123\"}},\"serviceCatalog\":[{\"

Re: [jclouds] Adds some easy-of-use for openstack mock tests (#224)

2013-12-09 Thread Zack Shoylev
> > + @SuppressWarnings("serial") It is, as in, there is a warning here otherwise. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/224/files#r8222185

Re: [jclouds] Allow the HeaderParam annotation to be used in a Caller. (#226)

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

Jenkins build became unstable: jclouds » jclouds #680

2013-12-09 Thread BuildHive
See

Re: [jclouds] JCLOUDS-356 multipart uploads with InputStream payloads (#192)

2013-12-09 Thread Eric Evans
> [minor] How big are the files? If not huuuge, we can also avoid the outFile > by something like: The files are 14MB. Mostly I did this because Guava's `Files.hash()` made the comparison terse. But it also has the benefit of leaving both the actual and expected files in `target/` should you

Re: [jclouds] JCLOUDS-356 multipart uploads with InputStream payloads (#192)

2013-12-09 Thread Eric Evans
> [minor] should be able to use a File payload here? Or do you need to set the > length for some reason (I think jclouds should do that internally too, > though?). At this point, it should be the equivalent, but the old code treated `File` and `InputStream` differently (`File` worked, and `Inpu

Re: [jclouds] Allow the HeaderParam annotation to be used in a Caller. (#226)

2013-12-09 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #451](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/451/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/226#issuecomment-30197372

Re: [jclouds] Allow the HeaderParam annotation to be used in a Caller. (#226)

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

Re: [jclouds] Adds some easy-of-use for openstack mock tests (#224)

2013-12-09 Thread Everett Toews
> +* Ensures the request is json and has the same contents as the resource > +* file provided. > +* > +* @param request > +* @param method > +* The request method (such as GET). > +* @param resourceLocation > +* The location of the resource file.

Re: [jclouds] Adds some easy-of-use for openstack mock tests (#224)

2013-12-09 Thread Everett Toews
> + public void assertAuthentication(MockWebServer server) { > + assertEquals(server.getRequestCount(), 2); > + try { > + assertEquals(server.takeRequest().getRequestLine(), "POST /tokens > HTTP/1.1"); > + } catch (InterruptedException e) { > + Throwables.propagate

Re: [jclouds] Adds some easy-of-use for openstack mock tests (#224)

2013-12-09 Thread Everett Toews
> > + @SuppressWarnings("serial") This SuppressWarning doesn't seem to be necessary. Is it? --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/224/files#r8221513

Re: [jclouds] Adds some easy-of-use for openstack mock tests (#224)

2013-12-09 Thread Everett Toews
> @@ -96,4 +112,90 @@ public void enqueueResponse(MockResponse response) { > >return dispatcher; > } > + > + /** > +* Ensure json headers are included > +* */ > + public MockResponse addJsonHeaders(MockResponse mockResponse) { addJsonHeaders makes it sound like the values

Re: [jclouds] Allowing Guava MediaType for Content-Type (#225)

2013-12-09 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #450](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/450/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/225#issuecomment-30196682

Re: [jclouds] Allowing Guava MediaType for Content-Type (#225)

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

Re: [jclouds] Adds some easy-of-use for openstack mock tests (#224)

2013-12-09 Thread Everett Toews
> public A api(String uri, String provider, Properties overrides) { >overrides.setProperty(PROPERTY_MAX_RETRIES, "1"); > > - return ContextBuilder.newBuilder(provider) > -.credentials("jclouds:joe", "letmein") > -.endpoint(uri) > -.overrides(ov

Re: [jclouds] Adds some easy-of-use for openstack mock tests (#224)

2013-12-09 Thread Everett Toews
> > - private final Set modules = ImmutableSet. of( > - new ExecutorServiceModule(sameThreadExecutor(), > sameThreadExecutor())); > + private final Set modules = ImmutableSet. of(new > ExecutorServiceModule(sameThreadExecutor(), Please preserve the formatting here. --- Reply to th

Re: [jclouds] Adds some easy-of-use for openstack mock tests (#224)

2013-12-09 Thread Everett Toews
> */ > public class BaseOpenStackMockTest { > > - public static final String accessRackspace = > "{\"access\":{\"token\":{\"id\":\"b84f4a37-5126-4603-9521-ccd0665fbde1\",\"expires\":\"2013-04-13T16:49:57.000-05:00\",\"tenant\":{\"id\":\"123123\",\"name\":\"123123\"}},\"serviceCatalog\":[{\"

Re: [jclouds] Adds some easy-of-use for openstack mock tests (#224)

2013-12-09 Thread Everett Toews
> +* The name of the resource. > +* @return The content of the resource > +*/ > + public String stringFromResource(String resourceName) { > + try { > + return > Strings2.toStringAndClose(getClass().getResourceAsStream(resourceName)); > + } catch (IOExcepti

Re: [jclouds] [JCLOUDS-263] Usage of internalURL services' endpoints (#142)

2013-12-09 Thread Everett Toews
@demobox @sallum I got my the test to fail so everything is looking good! :D Which is to say, I get the error HttpResponseException: not.even.an.ip connecting to GET http://not.even.an.ip/v2/134e0fd0c239484dbfe3fdcb5211d4dc/servers When I ran the test against my DevStack with the internalUR

Re: [jclouds] [JCLOUDS-263] Usage of internalURL services' endpoints (#142)

2013-12-09 Thread Everett Toews
> +import com.google.inject.AbstractModule; > +import com.google.inject.Module; > +import com.google.inject.Singleton; > + > +/** > + * Simple live test to check the correct loading of the internal endpoint > + * services. > + * > + * @author Ignacio Mulas > + * > + */ > +@Test(groups = "live", t

[jclouds] Allow the HeaderParam annotation to be used in a Caller. (#226)

2013-12-09 Thread Everett Toews
This allows jclouds to factor out common headers into the Caller so they don't have to be repeated in the Callee. You can merge this Pull Request by running: git pull https://github.com/rackspace/jclouds caller-header-param Or you can view, comment on it, or merge it online at: https://gith

Re: [jclouds] Make BlobName Function type-safe (#38)

2013-12-09 Thread Andrew Phillips
Yay. Finally! Only JIRA, rebase'n'merge to go now... --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/38#issuecomment-30195194

Re: [jclouds] Allowing Guava MediaType for Content-Type (#225)

2013-12-09 Thread Andrew Phillips
> jclouds-pull-requests #447 FAILURE They say: "sucks boo", it seems ;-) Clojure fix coming up... --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/225#issuecomment-30195108

Re: [jclouds] Make BlobName Function type-safe (#38)

2013-12-09 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #449](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/449/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/38#issuecomment-30194791

Re: [jclouds] Make BlobName Function type-safe (#38)

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

Re: [jclouds] Make BlobName Function type-safe (#38)

2013-12-09 Thread Andrew Phillips
> jclouds-pull-requests #446 UNSTABLE Egads! Not **again**. [This one](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/org.apache.jclouds$jclouds-compute/446/testReport/junit/org.jclouds.compute.util/ConcurrentOpenSocketFinderTest/testChecksSocketsConcurrently/) also looks weird, thou

Re: [jclouds] Make BlobName Function type-safe (#38)

2013-12-09 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #446](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/446/) 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/38#issuecomment-30191624

Re: [jclouds] Make BlobName Function type-safe (#38)

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

Re: [jclouds] Allowing Guava MediaType for Content-Type (#225)

2013-12-09 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #447](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/447/) 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/pull/225#issuecomment-30190916

Re: [jclouds] Allowing Guava MediaType for Content-Type (#225)

2013-12-09 Thread CloudBees pull request builder plugin
[jclouds-java-7-pull-requests #910](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/910/) 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/pull/225#issuecomment-30190884

[jclouds] Allowing Guava MediaType for Content-Type (#225)

2013-12-09 Thread Andrew Phillips
Alternative version of #44 You can merge this Pull Request by running: git pull https://github.com/jclouds/jclouds pr-44-alternative Or you can view, comment on it, or merge it online at: https://github.com/jclouds/jclouds/pull/225 -- Commit Summary -- * Allowing Guava MediaType for Con

Re: [jclouds] Allowing Guava MediaType for Content-Type (#225)

2013-12-09 Thread Andrew Phillips
Let's see what the PR builders say! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/225#issuecomment-30190231

Re: [jclouds] Make BlobName Function type-safe (#38)

2013-12-09 Thread Andrew Phillips
> jclouds-java-7-pull-requests #908 UNSTABLE Weird [failure](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/org.apache.jclouds.common$azure-common/908/testReport/junit/org.jclouds.azure.storage.filters/SharedKeyLiteAuthenticationTest/testIdempotent/). Not obviously related, bu

Re: [jclouds] Make BlobName Function type-safe (#38)

2013-12-09 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #445](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/445/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/38#issuecomment-30189156

Re: [jclouds] Make BlobName Function type-safe (#38)

2013-12-09 Thread CloudBees pull request builder plugin
[jclouds-java-7-pull-requests #908](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/908/) 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/38#issuecomment-30189094

Re: Open PRs

2013-12-09 Thread Andrew Phillips
I've added comments to all old PRs I've found. Let's see if we get some feedback and can merge/close! :) Thanks, Ignasi! ap

Re: [jclouds] Deprecate BlobBuilder.calculateMD5 (#223)

2013-12-09 Thread Andrew Phillips
> @@ -121,6 +121,7 @@ > * > * @see Payloads#calculateMD5 > */ > + @Deprecated Use the [`@deprecated` Javadoc tag](http://docs.oracle.com/javase/1.5.0/docs/guide/javadoc/deprecation/deprecation.html#javadoc_tag)? Renders more nicely in the Javadoc, then. Happy to am

Re: [jclouds] Adds some easy-of-use for openstack mock tests (#224)

2013-12-09 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #444](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/444/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/224#issuecomment-30187833

Re: [jclouds] Adds some easy-of-use for openstack mock tests (#224)

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

Re: [jclouds] JCLOUDS-32: Using less magic in route53 signature (#222)

2013-12-09 Thread Andrew Phillips
Committed to [master](https://git-wip-us.apache.org/repos/asf?p=jclouds.git;a=commit;h=acde2beff1d04301af69a68ce598d7d3a96a92f0). Any fixes we want to make to avoid potential OOMEs can be added in a separate PR, I think. --- Reply to this email directly or view it on GitHub: https://github.com/

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

2013-12-09 Thread BuildHive
See

Re: [jclouds] JCLOUDS-356 multipart uploads with InputStream payloads (#192)

2013-12-09 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #443](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/443/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/192#issuecomment-30187498

Re: [jclouds] JCLOUDS-356 multipart uploads with InputStream payloads (#192)

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

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

2013-12-09 Thread CloudBees pull request builder plugin
[jclouds-site-pull-requests #103](https://jclouds.ci.cloudbees.com/job/jclouds-site-pull-requests/103/) 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-site/pull/32#issuecomment-30187319

Re: [jclouds] Delete objects in a container efficiently. (#214)

2013-12-09 Thread Andrew Phillips
Thanks for taking the time to review, @andrewgaul! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/214#issuecomment-30187292

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

2013-12-09 Thread Zack Shoylev
> Something weird going on with the numbering now: I could not figure that out... --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-site/pull/32#issuecomment-30187210

Re: [jclouds] Allow Guava MediaType for Content-Type (#44)

2013-12-09 Thread Andrew Phillips
> or is still some investigation pending regarding the test? I still think `MediaType` should be non-nullable. Ideally, I'd like to try a variant of this PR with a `checkNotNull` to see if that works too. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/p

Re: [jclouds] Remove BlobBuilder.calculateMD5 (#218)

2013-12-09 Thread Andrew Phillips
Still the same [clojure failure](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/441/console) by the way, it seems: ``` org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal com.theoryinpractise:clojure-maven-plugin:1.3.10:test (test-clojure) on project jclou

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

2013-12-09 Thread BuildHive
See

Re: [jclouds] Date code for AWS cannot handle iso8601 dates without milliseconds. (#105)

2013-12-09 Thread Florian Leibert
Sorry guys - I haven't had a chance to look at it and won't be able to in the near future :/ Thanks again. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/105#issuecomment-30186866

Re: [jclouds] Make BlobName Function type-safe (#38)

2013-12-09 Thread Andrew Phillips
> +1 too. It's been 6 months, but can we merge this one? Quick JIRA issue & merge? Will close'n'reopen now to re-trigger the PR builders. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/38#issuecomment-30186748

Re: [jclouds] Make BlobName Function type-safe (#38)

2013-12-09 Thread Andrew Phillips
Closed #38. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/38

Re: [jclouds-labs-openstack] Sample refactoring of GroupApiExpectTest to use MockWebServer (#59)

2013-12-09 Thread Zack Shoylev
Build failure is expected. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-openstack/pull/59#issuecomment-30186049

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

2013-12-09 Thread Zack Shoylev
Half an hour of cleanup. I will have a look a bit later tonight. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-site/pull/32#issuecomment-30185978

Re: [jclouds] JCLOUDS-356 multipart uploads with InputStream payloads (#192)

2013-12-09 Thread Eric Evans
@andrewgaul > Can you write this with Guava ByteStreams.limit? This avoids the extra copy > in memory. This might be doable, but I'm not sure how it would look. It's not enough to simply return a wrapped `InputStream`, because you still need a way of paging through the streamed data. You could

Re: [jclouds] Deprecate BlobBuilder.calculateMD5 (#223)

2013-12-09 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #442](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/442/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/223#issuecomment-30185385

Re: [jclouds-labs-openstack] Sample refactoring of GroupApiExpectTest to use MockWebServer (#59)

2013-12-09 Thread CloudBees pull request builder plugin
[jclouds-labs-openstack-pull-requests #108](https://jclouds.ci.cloudbees.com/job/jclouds-labs-openstack-pull-requests/108/) 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/59

Re: [jclouds-labs-openstack] Sample refactoring of GroupApiExpectTest to use MockWebServer (#59)

2013-12-09 Thread BuildHive
[jclouds » jclouds-labs-openstack #692](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-labs-openstack/692/) 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: ht

Re: [jclouds-karaf] [JCLOUDS-189] Upgrade to Karaf 2.3.2. Explicitly specify jclouds feature... (#24)

2013-12-09 Thread Andrew Gaul
Fixing this would include https://issues.apache.org/jira/browse/KARAF-2368";>KARAF-2368 and address a major usability issue with jclouds and karaf. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-karaf/pull/24#issuecomment-30185148

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

2013-12-09 Thread BuildHive
See -- [...truncated 1870 lines...] location: class GroupApiMockTest

Re: [jclouds] JCLOUDS-356 multipart uploads with InputStream payloads (#192)

2013-12-09 Thread Zack Shoylev
+1 and thanks for the live test --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/192#issuecomment-30185159

Re: [jclouds] Deprecate BlobBuilder.calculateMD5 (#223)

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

Re: Open PRs

2013-12-09 Thread Ignasi Barrera
I've added comments to all old PRs I've found. Let's see if we get some feedback and can merge/close! :) On 9 December 2013 00:11, Ignasi Barrera wrote: > +1 to ping 'em all and see which of them are alive or may have activity. > I'll have a look tomorrow (CET :)). > > I'm not sure about when to

Re: [jclouds-karaf] JCLOUDS-85: Improving the creation of instance of PropertyShellTableFact... (#18)

2013-12-09 Thread Ignasi Barrera
So, what is he state of this PR? Is there more work to do here? --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-karaf/pull/18#issuecomment-30184911

Re: [jclouds-karaf] [JCLOUDS-189] Upgrade to Karaf 2.3.2. Explicitly specify jclouds feautre... (#24)

2013-12-09 Thread Ignasi Barrera
Just a ping... Is there any expectation to get this PR working? --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-karaf/pull/24#issuecomment-30184779

Re: [jclouds] Adds some easy-of-use for openstack mock tests (#224)

2013-12-09 Thread Zack Shoylev
Example usage: https://github.com/jclouds/jclouds-labs-openstack/pull/59 --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/224#issuecomment-30184672

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

2013-12-09 Thread Ignasi Barrera
@zack-shoylev @demobox what is still pending to have this merged? --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-site/pull/32#issuecomment-30184636

Re: [jclouds-labs-openstack] Sample refactoring of GroupApiExpectTest to use MockWebServer (#59)

2013-12-09 Thread Zack Shoylev
Depends on https://github.com/jclouds/jclouds/pull/224 --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-openstack/pull/59#issuecomment-30184651

[jclouds-labs-openstack] Sample refactoring of GroupApiExpectTest to use MockWebServer (#59)

2013-12-09 Thread Zack Shoylev
This approach could/should be applied to the other expect tests. You can merge this Pull Request by running: git pull https://github.com/rackerlabs/jclouds-labs-openstack mock-webserver-refactor Or you can view, comment on it, or merge it online at: https://github.com/jclouds/jclouds-labs-o

[jclouds] Adds some easy-of-use for openstack mock tests (#224)

2013-12-09 Thread Zack Shoylev
You can merge this Pull Request by running: git pull https://github.com/rackspace/jclouds refactor-base-mock-tests Or you can view, comment on it, or merge it online at: https://github.com/jclouds/jclouds/pull/224 -- Commit Summary -- * Adds some easy-of-use for openstack mock tests --

Re: [jclouds-karaf] Remove --string-payload and --url-payload flags (#21)

2013-12-09 Thread Andrew Gaul
Closed #21. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-karaf/pull/21

Re: [jclouds-karaf] Remove --string-payload and --url-payload flags (#21)

2013-12-09 Thread Andrew Gaul
Closing this out, not compatible with karaf. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-karaf/pull/21#issuecomment-30184539

Re: [jclouds-labs-openstack] Replacing strange chars in pom.xml with spaces (#53)

2013-12-09 Thread Ignasi Barrera
@demobox Can this be merged? --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-openstack/pull/53#issuecomment-30184481

Re: [jclouds-karaf] Allow stdin local file for streaming writes (#23)

2013-12-09 Thread Ignasi Barrera
Can this be merged? --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-karaf/pull/23#issuecomment-30184297

Re: [jclouds-karaf] [JCLOUDS-225] Configured itests to use the generated repo. (#25)

2013-12-09 Thread Ignasi Barrera
Reopen it if still makes sense to have it open :) --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-karaf/pull/25#issuecomment-30184239

Re: [jclouds] fixing jclouds-120 (#30)

2013-12-09 Thread Ignasi Barrera
@tangyong @everett-toews Any update on how this should be fixed/continued? --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/30#issuecomment-30184141

Re: [jclouds] Jclouds 136 -- Implementation of DynECT HTTP Redirect service API (#37)

2013-12-09 Thread Ignasi Barrera
@sunnygleason It's been 5 months now. Is there any chance it gets some progress regarding the review comments? (No pressure, we are just trying to filter dead PRs). --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/37#issuecomment-30184007

Re: [jclouds] Allow Guava MediaType for Content-Type (#44)

2013-12-09 Thread Ignasi Barrera
So, can this be merged, or is still some investigation pending regarding the test? --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/44#issuecomment-30183857

  1   2   >