Re: [jclouds/jclouds] Increases session interval from 60 seconds to a higher value (#1206)

2018-05-11 Thread Ignasi Barrera
Closed #1206. -- 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/1206#event-1622909006

Re: [jclouds/jclouds] Increases session interval from 60 seconds to a higher value (#1206)

2018-05-11 Thread Ignasi Barrera
>It should be safe to increase this value in the provider metadata. 5 minutes >looks good but open to discuss Looks like a sane default. Merged to [master](http://git-wip-us.apache.org/repos/asf/jclouds/commit/a08de5b2) and

Re: [jclouds/jclouds] Increases session interval from 60 seconds to a higher value (#1206)

2018-05-11 Thread Daniel Estévez
Sorry about that, fixed -- 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/1206#issuecomment-388499413

Re: [jclouds/jclouds] Increases session interval from 60 seconds to a higher value (#1206)

2018-05-11 Thread Ignasi Barrera
The build failed. It looks like there's a missing static import. -- 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/1206#issuecomment-388498018

Re: [jclouds/jclouds] Increases session interval from 60 seconds to a higher value (#1206)

2018-05-11 Thread Ignasi Barrera
>Would be good to add this also to >https://github.com/danielestevez/jclouds/blob/83c0a3c7b255ec744c6150ce76c40e8301280c79/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/internal/AzureLiveTestUtils.java > ? That class just adds *overrides* for default provider properties,

[jclouds/jclouds] Increases session interval from 60 seconds to a higher value (#1206)

2018-05-11 Thread Daniel Estévez
To avoid problems with the cache, and specifically running the AzureComputeImageExtensionLiveTest suite. ARM is quite slow and it takes usually more than one minute (default value) for certain operations. The testImageIsCachedAfterBeingCreated in that suite fails often since it takes longer

Re: [jclouds/jclouds] Fixes Run SSH script for passwords with special characters (#1205)

2018-05-11 Thread Ignasi Barrera
Closed #1205. -- 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/1205#event-1622778189

Re: [jclouds/jclouds] Fixes Run SSH script for passwords with special characters (#1205)

2018-05-11 Thread Ignasi Barrera
Pushed to [master](http://git-wip-us.apache.org/repos/asf/jclouds/commit/82289948). @andreaturli I think the change is safe enough to cherry-pick it to 2.1.x, WDYT? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [jclouds/jclouds] Fixes Run SSH script for passwords with special characters (#1205)

2018-05-11 Thread Daniel Estévez
It's just a sometimes since these kind of characters can be included in password random generator https://github.com/danielestevez/jclouds/blob/9fef6ed06b027c1525579c88b58aa9f5833687fd/core/src/main/java/org/jclouds/util/PasswordGenerator.java#L65 -- You are receiving this because you are

Re: [jclouds/jclouds] Fixes Run SSH script for passwords with special characters (#1205)

2018-05-11 Thread Andrea Turli
Haven't tested myself but lgtm, thanks @danielestevez -- 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/1205#issuecomment-388466244

[jclouds/jclouds] Fixes Run SSH script for passwords with special characters (#1205)

2018-05-11 Thread Daniel Estévez
Passwords can contain special characters as parentheses, password parameter should be wrapped in '' to avoid this syntax errors in SSH command You can view, comment on, or merge this pull request online at: https://github.com/jclouds/jclouds/pull/1205 -- Commit Summary -- * Fixes Run SSH

Re: [jclouds/jclouds-site] JCLOUDS-1414: Keystone authentication improvment (#216)

2018-05-11 Thread jclouds-commentator
Go to http://9ee4695ccebd96b41334-2a21ba0b2756ec3f93728892323454a7.r87.cf5.rackcdn.com/ to review your changes. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[jclouds/jclouds-site] JCLOUDS-1414: Keystone authentication improvment (#216)

2018-05-11 Thread Alix Lourme
Following #214, some OpenStack keystone authentication improvments, following @demobox review. You can view, comment on, or merge this pull request online at: https://github.com/jclouds/jclouds-site/pull/216 -- Commit Summary -- * JCLOUDS-1414: Keystone authentication improvment -- File

Re: [jclouds/jclouds-site] Documentation about Openstack Keystone v2/v3 usage (#214)

2018-05-11 Thread Andrew Phillips
> I will do it in a // PR (this one merged) after feedback on my remarks. Thanks for taking a look! I should add that I think most of my comments are minor, and it's also fine to leave things as they are unless we feel there's really a good reason for a follow-up PR. And thanks for helping

Re: [jclouds/jclouds-site] Documentation about Openstack Keystone v2/v3 usage (#214)

2018-05-11 Thread Alix Lourme
@demobox : Thanks for the review. I will do it in a // PR (this one merged) after feedback on my remarks. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [jclouds/jclouds-site] Documentation about Openstack Keystone v2/v3 usage (#214)

2018-05-11 Thread Alix Lourme
axel3rd commented on this pull request. > +{% highlight java %} +final Properties overrides = new Properties(); +overrides.put(KeystoneProperties.KEYSTONE_VERSION, "2"); +overrides.put(KeystoneProperties.TENANT_NAME, "myTenant"); + +ContextBuilder.newBuilder("openstack-nova") +

Re: [jclouds/jclouds-site] Documentation about Openstack Keystone v2/v3 usage (#214)

2018-05-11 Thread Alix Lourme
axel3rd commented on this pull request. > @@ -92,6 +93,220 @@ There are some differences in terminology between jclouds > and OpenStack that sho +## Keystone v2-v3 authentication + +Openstack Keystone (aka: [OpenStack Identity Service](https://docs.openstack.org/keystone/latest/)) has