Re: [jclouds/jclouds] [JCLOUDS-1422] Ensure that LocalBlobStore.list honors !recursive even when a prefix is defined (#1212)

2018-05-22 Thread Andrew Gaul
gaul commented on this pull request. > @@ -72,7 +72,7 @@ public void testListWithPrefix() { blobStore.putBlob(containerName, blobStore.blobBuilder("bar").payload("").build()); Iterable results = concatter.execute(containerName, -ListContainerOptions.Builder.prefix(p

[jclouds/jclouds] [JCLOUDS-1422] Ensure that LocalBlobStore.list honors !recursive even when a prefix is defined (#1212)

2018-05-22 Thread Jesse Glick
[JCLOUDS-1422](https://issues.apache.org/jira/browse/JCLOUDS-1422) Amends #788. FYI @carlossg: encountered during https://github.com/jenkinsci/artifact-manager-s3-plugin/pull/28 You can view, comment on, or merge this pull request online at: https://github.com/jclouds/jclouds/pull/1212 -- Co

[jira] [Created] (JCLOUDS-1422) LocalBlobStore.list ignores recursive flag when prefix set

2018-05-22 Thread Jesse Glick (JIRA)
Jesse Glick created JCLOUDS-1422: Summary: LocalBlobStore.list ignores recursive flag when prefix set Key: JCLOUDS-1422 URL: https://issues.apache.org/jira/browse/JCLOUDS-1422 Project: jclouds

Re: [jclouds/jclouds] [JCLOUDS-1200] Mark nonserializable fields in HttpResponseException and subtypes as transient (#1211)

2018-05-22 Thread Andrew Gaul
I periodically run SpotBugs via `mvn spotbugs:check` but we configure it in `project/pom.xml`. I do not believe that the default configuration includes the `SE_BAD_FIELD` check but I could not figure out how to enable it. https://github.com/spotbugs/spotbugs/issues/385#issuecomment-352146321 s

Re: [jclouds/jclouds-labs] Upgrade to java8 (#436)

2018-05-22 Thread Andrea Turli
sure thanks for double-checking it -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/pull/436#issuecomment-391117526

Re: [jclouds/jclouds-labs] Upgrade to java8 (#436)

2018-05-22 Thread Andrew Phillips
> makes sense? Yes, clear, thanks. Just wanted to make sure we weren't missing any code that we were intending to add ;-) -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/pull/436#issu

Re: [jclouds/jclouds-labs] Upgrade to java8 (#436)

2018-05-22 Thread Andrea Turli
@demobox I think guava21 updates are incidentally required as we updated guava21 in jclouds/jclouds, so technically the PR doesn't bump guava version for jclouds-labs in its pom.xml but it happens because of the upstream changes. makes sense? -- You are receiving this because you are subscribe

Re: [jclouds/jclouds-labs] Upgrade to java8 (#436)

2018-05-22 Thread Andrea Turli
andreaturli commented on this pull request. > @@ -40,7 +40,7 @@ import java.util.Properties; import java.util.Set; -import static com.google.common.util.concurrent.MoreExecutors.sameThreadExecutor; +import static com.google.common.util.concurrent.MoreExecutors.newDirectExecutorService; @d

Re: [jclouds/jclouds-labs] Upgrade to java8 (#436)

2018-05-22 Thread Andrew Phillips
> bump guava version to 21.0 @andreaturli Just to clarify: is this part of this PR? I didn't see any code change related to that, but it may have been removed or after an underlying merge. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view

[jclouds/jclouds-examples] [do not merge] bump to jdk8 (#93)

2018-05-22 Thread Andrea Turli
pre-validation PR waiting for a new jclouds release based on jdk8 You can view, comment on, or merge this pull request online at: https://github.com/jclouds/jclouds-examples/pull/93 -- Commit Summary -- * bump to jdk8 -- File Changes -- M blobstore-basics/pom.xml (9) M blobstore-la

Jenkins build is back to normal : jclouds-labs-2.1.x-branch » jclouds joyent-cloudapi api #32

2018-05-22 Thread jenkins-no-reply
See

Jenkins build is back to normal : jclouds-labs-2.1.x-branch #32

2018-05-22 Thread jenkins-no-reply
See

Re: [jclouds/jclouds-karaf] upgrade to java 8 (#108)

2018-05-22 Thread Andrea Turli
I think `maven-enforcer-plugin` config comes from jclouds/jclouds, do we want to override only for `jclouds/jclouds-jaraf` ? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-karaf/pull/108#

Re: [jclouds/jclouds-karaf] upgrade to java 8 (#108)

2018-05-22 Thread Ignasi Barrera
According to [this](http://karaf.922171.n3.nabble.com/compilation-failing-with-karaf-4-2-0-M2-td4052701.html) it looks like that Maven >= 3.3.x is needed for the `karaf-maven-plugin` which we use to validate the features can be installed. I'd suggest configuring the `maven-enforcer-plugin` in th

Build failed in Jenkins: jclouds-labs-2.1.x-branch #31

2018-05-22 Thread jenkins-no-reply
See Changes: [Ignasi Barrera] JCLOUDS-1421: Add default credentials to images created by the -- [...truncated 1.22 MB...] [JENKINS] Recording test results [

Build failed in Jenkins: jclouds-labs-2.1.x-branch » jclouds joyent-cloudapi api #31

2018-05-22 Thread jenkins-no-reply
See Changes: [Ignasi Barrera] JCLOUDS-1421: Add default credentials to images created by the -- [INFO]

Re: [jclouds/jclouds-labs] Upgrade to java8 (#436)

2018-05-22 Thread Andrea Turli
merged at [master](http://git-wip-us.apache.org/repos/asf/jclouds-labs/commit/1b5dd9fe) -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/pull/436#issuecomment-391017677

Re: [jclouds/jclouds-labs] Upgrade to java8 (#436)

2018-05-22 Thread Andrea Turli
Closed #436. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/pull/436#event-1639523209

Re: [jclouds/jclouds-karaf] upgrade to java 8 (#108)

2018-05-22 Thread Ignasi Barrera
rebuild please -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-karaf/pull/108#issuecomment-391017105

Re: [jclouds/jclouds-labs] Upgrade to java8 (#436)

2018-05-22 Thread Andrea Turli
cool, thanks @nacx and @gaul merging now -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/pull/436#issuecomment-391014797

Re: [jclouds/jclouds-labs] Upgrade to java8 (#436)

2018-05-22 Thread Ignasi Barrera
nacx approved this pull request. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/pull/436#pullrequestreview-122199296

Re: [jclouds/jclouds] Add default credentials to images created by the ImageExtension (#1210)

2018-05-22 Thread Ignasi Barrera
Merged t [master](http://git-wip-us.apache.org/repos/asf/jclouds/commit/b76a594e) ([labs](http://git-wip-us.apache.org/repos/asf/jclouds-labs/commit/331fd685)) and [2.1.x](http://git-wip-us.apache.org/repos/asf/jclouds/commit/3878e6a6) ([labs](http://git-wip-us.apache.org/repos/asf/jclouds-labs

Re: [jclouds/jclouds] Add default credentials to images created by the ImageExtension (#1210)

2018-05-22 Thread Ignasi Barrera
Closed #1210. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1210#event-1639491145

[jira] [Resolved] (JCLOUDS-1421) Images created by the ImageExtension may lack the mandatory default credentials

2018-05-22 Thread Ignasi Barrera (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-1421?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ignasi Barrera resolved JCLOUDS-1421. - Resolution: Fixed Fix Version/s: 2.1.1 2.2.0 > Images created

[jira] [Commented] (JCLOUDS-1421) Images created by the ImageExtension may lack the mandatory default credentials

2018-05-22 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-1421?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16484057#comment-16484057 ] ASF subversion and git services commented on JCLOUDS-1421: -- Com

[jira] [Commented] (JCLOUDS-1421) Images created by the ImageExtension may lack the mandatory default credentials

2018-05-22 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-1421?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16484058#comment-16484058 ] ASF subversion and git services commented on JCLOUDS-1421: -- Com

[jira] [Commented] (JCLOUDS-1421) Images created by the ImageExtension may lack the mandatory default credentials

2018-05-22 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-1421?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16484053#comment-16484053 ] ASF subversion and git services commented on JCLOUDS-1421: -- Com

[jira] [Commented] (JCLOUDS-1421) Images created by the ImageExtension may lack the mandatory default credentials

2018-05-22 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/JCLOUDS-1421?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16484051#comment-16484051 ] ASF subversion and git services commented on JCLOUDS-1421: -- Com

Re: [jclouds/jclouds] [JCLOUDS-1200] Mark nonserializable fields in HttpResponseException and subtypes as transient (#1211)

2018-05-22 Thread Jesse Glick
> I investigated configuring FindBugs and SpotBugs to find this error but could > not figure it out. I see some FindBugs annotations in the project, but no FindBugs mojo being run. Something you would run manually? The detector in question would be [`SE_BAD_FIELD`](http://findbugs.sourceforge.