Re: [jclouds/jclouds] More fixes to parallel download resource cleanup (#1010)

2016-09-14 Thread Zack Shoylev
merged

-- 
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/1010#issuecomment-247129173

Re: [jclouds/jclouds] More fixes to parallel download resource cleanup (#1010)

2016-09-14 Thread Zack Shoylev
Closed #1010.

-- 
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/1010#event-789701771

Re: [jclouds/jclouds] More fixes to parallel download resource cleanup (#1010)

2016-09-14 Thread Andrew Gaul
andrewgaul 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/pull/1010#pullrequestreview-29746

[jira] [Commented] (JCLOUDS-1166) Remove uses of the 'com.google.gson.internal' package

2016-09-14 Thread Ignasi Barrera (JIRA)

[ 
https://issues.apache.org/jira/browse/JCLOUDS-1166?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15491100#comment-15491100
 ] 

Ignasi Barrera commented on JCLOUDS-1166:
-

[~andrewp] what do you think about the custom bundle approach?

> Remove uses of the 'com.google.gson.internal' package
> -
>
> Key: JCLOUDS-1166
> URL: https://issues.apache.org/jira/browse/JCLOUDS-1166
> Project: jclouds
>  Issue Type: Bug
>  Components: jclouds-chef, jclouds-core
>Affects Versions: 1.9.2
>Reporter: Ignasi Barrera
>  Labels: gson
>
> Starting from Gson 2.6, the {{com.google.gson.internal}} packages are no 
> longer exported in the OSGi bundles. This makes it impossible to use jclouds 
> in an OSGi environment if upgrading to such versions of Gson.
> There is no change to add the exports back for that package (see [this 
> discussion|https://github.com/google/gson/pull/916]), so we should stop using 
> those classes.
> See also: http://markmail.org/message/olgebygfgdy3hwtm



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (JCLOUDS-1005) Backblaze B2 Cloud Storage

2016-09-14 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/JCLOUDS-1005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15491045#comment-15491045
 ] 

ASF subversion and git services commented on JCLOUDS-1005:
--

Commit 5e01c0733b9e5cb2937a46fbfd7fae257bec2687 in jclouds-cli's branch 
refs/heads/master from [~gaul]
[ https://git-wip-us.apache.org/repos/asf?p=jclouds-cli.git;h=5e01c07 ]

JCLOUDS-1005: Add Backblaze B2


> Backblaze B2 Cloud Storage
> --
>
> Key: JCLOUDS-1005
> URL: https://issues.apache.org/jira/browse/JCLOUDS-1005
> Project: jclouds
>  Issue Type: New Feature
>  Components: jclouds-blobstore
>Affects Versions: 1.9.1
>Reporter: Andrew Gaul
>Assignee: Andrew Gaul
>  Labels: b2
> Fix For: 2.0.0
>
>
> Backblaze announced B2 which has its own object storage API:
> https://www.backblaze.com/b2/docs/



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (JCLOUDS-1132) Amazon AWS - Create Volume - query params reported as unrecognized

2016-09-14 Thread Tusa Mihail Cristian (JIRA)

[ 
https://issues.apache.org/jira/browse/JCLOUDS-1132?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15490994#comment-15490994
 ] 

Tusa Mihail Cristian commented on JCLOUDS-1132:
---

Hi Felix,

You can check out the Cloud Slang open-source project on GitHub. In
https://github.com/CloudSlang/cs-actions/tree/master/cs-jclouds/src/main/java/io/cloudslang/content/jclouds/actions
you can find actions that can be (re)used. Basically they made HTTP calls
using Query API pattern against Amazon without using any third-party
library. So, check it out if you have time... maybe you can find there
something useful.

Note that right now is a mix (J-Clouds and Query API) but soon will be only
Query API.

Mihai Tusa




> Amazon AWS - Create Volume - query params reported as unrecognized
> --
>
> Key: JCLOUDS-1132
> URL: https://issues.apache.org/jira/browse/JCLOUDS-1132
> Project: jclouds
>  Issue Type: Bug
>  Components: jclouds-compute
>Affects Versions: 1.9.0, 1.9.2
> Environment: Amazon AWS EC2 management console
>Reporter: Tusa Mihail Cristian
>
> Let say that I want to create a Volume that is encrypted. So, I set the 
> encrypted on "true"... in request query params apear "=true". In 
> this case the error that I got is: 
> UnknownParameterThe parameter 
> Encrypted is not 
> recognizedc874434a-b00e-425d-baf4-18087441603a
>  
> instead it should be: "=1" as it is shown in Amazon docs: 
> http://docs.aws.amazon.com/AWSEC2/latest/APIReference/API_CreateVolume.html 
> in Example/Sample request. The problems are also related with Volume Type. If 
> not specified than creates a standard volume but if anything specified (any 
> of valid Values: standard | io1 | gp2 | sc1 | st1 ) the same error with 
> unrecognized param is received.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


Re: [jclouds/jclouds-karaf] JCLOUDS-1172: Use an OSGi-compliant way of loading JSR 223 script engines (#79)

2016-09-14 Thread Ignasi Barrera
Yay! Thanks @demobox!

-- 
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/79#issuecomment-247085174

Re: [jclouds/jclouds] More fixes to parallel download resource cleanup (#1010)

2016-09-14 Thread Ignasi Barrera
>I will try to do a separate PR for it (out of scope for this one).

Great :) Thanks!

-- 
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/1010#issuecomment-247084474

Re: [jclouds/jclouds] More fixes to parallel download resource cleanup (#1010)

2016-09-14 Thread Zack Shoylev
@nacx 1176: I will try to do a separate PR for it (out of scope for this one).

-- 
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/1010#issuecomment-247083691

Re: [jclouds/jclouds-karaf] JCLOUDS-1172: Use an OSGi-compliant way of loading JSR 223 script engines (#79)

2016-09-14 Thread Andrew Phillips
Merged to master: df7d4de

-- 
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/79#issuecomment-247083256

Re: [jclouds/jclouds-karaf] JCLOUDS-1172: Use an OSGi-compliant way of loading JSR 223 script engines (#79)

2016-09-14 Thread Andrew Phillips
Closed #79.

-- 
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/79#event-789475508

Re: [jclouds/jclouds-labs] fix azure-arm features live tests (#317)

2016-09-14 Thread Ignasi Barrera
Can you share the live test output after this change?
The base compute service has some issues regarding access to the VM. I will be 
able to allocate some time to look at it next week, so I wouldn't worry about 
those.

-- 
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/317#issuecomment-247082829

Re: [jclouds/jclouds-labs] fix azure-arm features live tests (#317)

2016-09-14 Thread Ignasi Barrera
Build failed due to checkstyle errors:
https://jclouds.ci.cloudbees.com/job/jclouds-labs-pull-requests/1447/org.apache.jclouds.labs$azurecompute-arm/violations/

-- 
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/317#issuecomment-247082444

Re: [jclouds/jclouds] More fixes to parallel download resource cleanup (#1010)

2016-09-14 Thread Ignasi Barrera
Does this also fix JCLOUDS-1176? (Just asking to understand the scope of this 
PR :))

-- 
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/1010#issuecomment-247080469

[jira] [Updated] (JCLOUDS-1172) Script engines are not loaded in the interactive shell

2016-09-14 Thread Andrew Phillips (JIRA)

 [ 
https://issues.apache.org/jira/browse/JCLOUDS-1172?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Andrew Phillips updated JCLOUDS-1172:
-
Attachment: JCLOUDS-1172.patch

> Script engines are not loaded in the interactive shell
> --
>
> Key: JCLOUDS-1172
> URL: https://issues.apache.org/jira/browse/JCLOUDS-1172
> Project: jclouds
>  Issue Type: Bug
>  Components: jclouds-karaf
>Affects Versions: 1.9.2
>Reporter: Ignasi Barrera
>Assignee: Andrew Phillips
> Fix For: 2.0.0
>
> Attachments: JCLOUDS-1172.patch
>
>
> When using the interactive shell, the script engines that are used to render 
> the values of the columns are not loaded, resulting in blank output lines 
> when running commands.
> This affects only the interactive shell.
> See this discussion for more details: 
> https://github.com/jclouds/jclouds-karaf/pull/77#issuecomment-244901813



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Resolved] (JCLOUDS-1172) Script engines are not loaded in the interactive shell

2016-09-14 Thread Andrew Phillips (JIRA)

 [ 
https://issues.apache.org/jira/browse/JCLOUDS-1172?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Andrew Phillips resolved JCLOUDS-1172.
--
   Resolution: Fixed
Fix Version/s: 2.0.0

> Script engines are not loaded in the interactive shell
> --
>
> Key: JCLOUDS-1172
> URL: https://issues.apache.org/jira/browse/JCLOUDS-1172
> Project: jclouds
>  Issue Type: Bug
>  Components: jclouds-karaf
>Affects Versions: 1.9.2
>Reporter: Ignasi Barrera
>Assignee: Andrew Phillips
> Fix For: 2.0.0
>
> Attachments: JCLOUDS-1172.patch
>
>
> When using the interactive shell, the script engines that are used to render 
> the values of the columns are not loaded, resulting in blank output lines 
> when running commands.
> This affects only the interactive shell.
> See this discussion for more details: 
> https://github.com/jclouds/jclouds-karaf/pull/77#issuecomment-244901813



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (JCLOUDS-1172) Script engines are not loaded in the interactive shell

2016-09-14 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/JCLOUDS-1172?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15490907#comment-15490907
 ] 

ASF subversion and git services commented on JCLOUDS-1172:
--

Commit df7d4de364e7ccf4288c50ff44616e0cd691a4ae in jclouds-karaf's branch 
refs/heads/master from [~demobox]
[ https://git-wip-us.apache.org/repos/asf?p=jclouds-karaf.git;h=df7d4de ]

JCLOUDS-1172: Use an OSGi-compliant way of loading JSR 223 script engines


> Script engines are not loaded in the interactive shell
> --
>
> Key: JCLOUDS-1172
> URL: https://issues.apache.org/jira/browse/JCLOUDS-1172
> Project: jclouds
>  Issue Type: Bug
>  Components: jclouds-karaf
>Affects Versions: 1.9.2
>Reporter: Ignasi Barrera
>Assignee: Andrew Phillips
>
> When using the interactive shell, the script engines that are used to render 
> the values of the columns are not loaded, resulting in blank output lines 
> when running commands.
> This affects only the interactive shell.
> See this discussion for more details: 
> https://github.com/jclouds/jclouds-karaf/pull/77#issuecomment-244901813



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


Re: [jclouds/jclouds-karaf] Added motivation for the bundles module (#80)

2016-09-14 Thread Andrew Phillips
Merged to master: 522e245. Thanks, Ignasi!

-- 
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/80#issuecomment-247077152

Re: [jclouds/jclouds-karaf] Added motivation for the bundles module (#80)

2016-09-14 Thread Andrew Phillips
Closed #80.

-- 
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/80#event-789447874

[jira] [Assigned] (JCLOUDS-1172) Script engines are not loaded in the interactive shell

2016-09-14 Thread Andrew Phillips (JIRA)

 [ 
https://issues.apache.org/jira/browse/JCLOUDS-1172?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Andrew Phillips reassigned JCLOUDS-1172:


Assignee: Andrew Phillips  (was: Ignasi Barrera)

> Script engines are not loaded in the interactive shell
> --
>
> Key: JCLOUDS-1172
> URL: https://issues.apache.org/jira/browse/JCLOUDS-1172
> Project: jclouds
>  Issue Type: Bug
>  Components: jclouds-karaf
>Affects Versions: 1.9.2
>Reporter: Ignasi Barrera
>Assignee: Andrew Phillips
>
> When using the interactive shell, the script engines that are used to render 
> the values of the columns are not loaded, resulting in blank output lines 
> when running commands.
> This affects only the interactive shell.
> See this discussion for more details: 
> https://github.com/jclouds/jclouds-karaf/pull/77#issuecomment-244901813



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (JCLOUDS-1005) Backblaze B2 Cloud Storage

2016-09-14 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/JCLOUDS-1005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15490847#comment-15490847
 ] 

ASF subversion and git services commented on JCLOUDS-1005:
--

Commit ccb5627659bb5895ef49b78db5052d2ff768069d in jclouds-karaf's branch 
refs/heads/fix-script-engine from [~gaul]
[ https://git-wip-us.apache.org/repos/asf?p=jclouds-karaf.git;h=ccb5627 ]

JCLOUDS-1005: Add Backblaze B2


> Backblaze B2 Cloud Storage
> --
>
> Key: JCLOUDS-1005
> URL: https://issues.apache.org/jira/browse/JCLOUDS-1005
> Project: jclouds
>  Issue Type: New Feature
>  Components: jclouds-blobstore
>Affects Versions: 1.9.1
>Reporter: Andrew Gaul
>Assignee: Andrew Gaul
>  Labels: b2
> Fix For: 2.0.0
>
>
> Backblaze announced B2 which has its own object storage API:
> https://www.backblaze.com/b2/docs/



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


Re: [jclouds/jclouds] More fixes to parallel download resource cleanup (#1010)

2016-09-14 Thread Zack Shoylev
@zack-shoylev pushed 1 commit.

c18747d  review fixes


-- 
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jclouds/jclouds/pull/1010/files/304c72c3e24e376cebbb98bd3b8f8ad6ecd72dac..c18747de0aeb80b3a6ab9eae5c6ceddf6c8e3a1a


Re: [jclouds/jclouds] More fixes to parallel download resource cleanup (#1010)

2016-09-14 Thread Zack Shoylev
> @@ -753,8 +770,24 @@ public Void call() {
> return null;
>  }
>   throw new RuntimeException("After " + retryCountLimit + " retries: 
> " + lastException);
> +  }
> +
> +  // JDK-4715154
> +  private void closeDirectBuffer(MappedByteBuffer mbb) {

logger is not static

-- 
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/1010/files/304c72c3e24e376cebbb98bd3b8f8ad6ecd72dac#r78770058

[jira] [Commented] (JCLOUDS-1005) Backblaze B2 Cloud Storage

2016-09-14 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/JCLOUDS-1005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15490659#comment-15490659
 ] 

ASF subversion and git services commented on JCLOUDS-1005:
--

Commit ccb5627659bb5895ef49b78db5052d2ff768069d in jclouds-karaf's branch 
refs/heads/master from [~gaul]
[ https://git-wip-us.apache.org/repos/asf?p=jclouds-karaf.git;h=ccb5627 ]

JCLOUDS-1005: Add Backblaze B2


> Backblaze B2 Cloud Storage
> --
>
> Key: JCLOUDS-1005
> URL: https://issues.apache.org/jira/browse/JCLOUDS-1005
> Project: jclouds
>  Issue Type: New Feature
>  Components: jclouds-blobstore
>Affects Versions: 1.9.1
>Reporter: Andrew Gaul
>Assignee: Andrew Gaul
>  Labels: b2
> Fix For: 2.0.0
>
>
> Backblaze announced B2 which has its own object storage API:
> https://www.backblaze.com/b2/docs/



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


Re: [jclouds/jclouds-labs] add support for whitelisting locations (#308)

2016-09-14 Thread Andrea Turli
> @@ -46,9 +44,9 @@ protected void bindErrorHandlers() {
> @Override
> protected void installLocations() {
>super.installLocations();
> -  bind(ImplicitLocationSupplier.class).
> -  to(OnlyLocationOrFirstRegionOptionallyMatchingRegionId.class).

I think `FirstRegion` as `ImplicitLocationSupplier` is a better choice here

-- 
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/308/files/961db5c145342971726063e25776e0011ab8ed36#r78753512

Re: [jclouds/jclouds-labs] add support for whitelisting locations (#308)

2016-09-14 Thread Andrea Turli
@andreaturli pushed 1 commit.

aaae684  revert ZONES to REGIONS


-- 
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/308/files/961db5c145342971726063e25776e0011ab8ed36..aaae68457526b198a2b30d7f648b4ef0b46ec359


Re: [jclouds/jclouds-labs] add support for whitelisting locations (#308)

2016-09-14 Thread Andrea Turli
> @@ -46,9 +44,9 @@ protected void bindErrorHandlers() {
> @Override
> protected void installLocations() {
>super.installLocations();
> -  bind(ImplicitLocationSupplier.class).
> -  to(OnlyLocationOrFirstRegionOptionallyMatchingRegionId.class).

@nacx I'm having issues in restoring 
`OnlyLocationOrFirstRegionOptionallyMatchingRegionId` as it fall-back to 
`providerUri`which is always the same for each region. Wrong supplier?

-- 
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/308/files/961db5c145342971726063e25776e0011ab8ed36#r78728598

Re: [jclouds/jclouds-labs] add support for whitelisting locations (#308)

2016-09-14 Thread Andrea Turli
ok thanks, I guess this one confused me 
https://github.com/jclouds/jclouds-labs/pull/308#issuecomment-239163128 :)

-- 
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/308#issuecomment-246973355

Re: [jclouds/jclouds-labs] add support for whitelisting locations (#308)

2016-09-14 Thread Ignasi Barrera
I don't think so. The last comments have not been addressed. We agreed that the 
change form region to zone was not good, so all that stuff needs to be reverted.

-- 
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/308#issuecomment-246971336

Re: [jclouds/jclouds-labs] add support for whitelisting locations (#308)

2016-09-14 Thread Andrea Turli
@nacx good to merge?

-- 
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/308#issuecomment-246959604