Even if usually the project and its parent have the same version, dependencies
are resolved using parent POM, so all internal dependencies should use the
parent version (that's the case for all other dependencies in the project).
You can view, comment on, or merge this pull request online at:
ht
olivierlemasle commented on this pull request.
> @@ -234,6 +234,13 @@ public String get() {
return authenticationMethods.get(credentialType);
}
+ @Provides
+ @Singleton
+ @Named("credentialsRetryable")
+ protected final boolean credentialsRetryable(Function
getAccess) {
@n
@nacx You're right, and I've added a commit to address that. It still binds to
`retryOnRenew` (we still have to use it for HTTP 408 errors), but I've modified
this RetryHandler to skip retrying in the case of a token-based authentication.
@zack-shoylev I've added some basic unit tests. Please te
@olivierlemasle pushed 1 commit.
8679f28 Disable retry for keystone token auth + unit tests
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jclouds/jclouds/pull/1009/files/901d897d79b6a3f0459d9f5e8c3593d01633f9b6..8679f282defaf91e90c04
Hi @zack-shoylev
Thanks for your comments.
Yes, I'm aware that the retry mechanisms will not work if the authentication is
performed with a token. However, in some cases, this authentication type can
still be useful, and if the token is invalid, an error should be raised.
Regarding the tests,
This pull request adds the possibility to use JClouds for OpenStack with a
pre-existing Keystone token.
Cf.
http://developer.openstack.org/api-ref/identity/v2/?expanded=authenticate-detail
To authenticate with a tenant name and a token, you can do:
```
Properties config = new Properties();
conf
Since commit 56e687f4979d, Linux line endings (LF) are enforced. But on
Windows, a common practice is to set core.autocrlf to 'auto', wich mean
that the local copy of the file has Windows line endings, whereas the
remote copy has Linux line endings (cf.
https://help.github.com/articles/dealing-wit
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds/pull/863
-- Commit Summary --
* Update CloudStack API metadata
-- File Changes --
M apis/cloudstack/src/main/java/org/jclouds/cloudstack/CloudStackApi.java
(4)
M
apis/cloudstack/sr