Re: [jclouds/jclouds] OpenStack Keystone V3 - different auth "domains" support (JCLOUDS-1414) (#1204)

2018-05-10 Thread Ignasi Barrera
Pushed to [master](http://git-wip-us.apache.org/repos/asf/jclouds/commit/053dfd01) and [2.1.x](http://git-wip-us.apache.org/repos/asf/jclouds/commit/93a805ca). Thanks @axel3rd! -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on

Re: [jclouds/jclouds] OpenStack Keystone V3 - different auth "domains" support (JCLOUDS-1414) (#1204)

2018-05-10 Thread Ignasi Barrera
Closed #1204. -- 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/1204#event-1619345086

Re: [jclouds/jclouds] OpenStack Keystone V3 - different auth "domains" support (JCLOUDS-1414) (#1204)

2018-05-09 Thread Alix Lourme
> I will do that (your reviews, thanks) this wednesday. About useTenant flag I > will be more explicit and add a unit test to prove the use case. Done, with `useTenant` renamed with [some explanation](https://github.com/jclouds/jclouds/pull/1204/files#diff-918e1e161307a179875339dfa9caceedR109),

Re: [jclouds/jclouds] OpenStack Keystone V3 - different auth "domains" support (JCLOUDS-1414) (#1204)

2018-05-08 Thread Alix Lourme
@nacx : I will do that (your reviews, thanks) this wednesday. About `useTenant` flag I will be more explicit and add a unit test to prove the use case. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [jclouds/jclouds] OpenStack Keystone V3 - different auth "domains" support (JCLOUDS-1414) (#1204)

2018-05-08 Thread Alix Lourme
axel3rd commented on this pull request. >} else if (DOMAIN.equals(parts[0])) { return DomainScope.create(Name.create(parts[1])); } else { return DomainIdScope.create(Id.create(parts[1])); } } + + private Object

Re: [jclouds/jclouds] OpenStack Keystone V3 - different auth "domains" support (JCLOUDS-1414) (#1204)

2018-05-08 Thread Ignasi Barrera
nacx approved this pull request. Just some small comments. Thanks for the contribution @axel3rd! >} else if (DOMAIN.equals(parts[0])) { return DomainScope.create(Name.create(parts[1])); } else { return DomainIdScope.create(Id.create(parts[1])); } } +

[jclouds/jclouds] OpenStack Keystone V3 - different auth "domains" support (JCLOUDS-1414) (#1204)

2018-05-07 Thread Alix Lourme
Cf. [JCLOUDS-1414](https://issues.apache.org/jira/browse/JCLOUDS-1414) You can view, comment on, or merge this pull request online at: https://github.com/jclouds/jclouds/pull/1204 -- Commit Summary -- * OpenStack Keystone V3 - different auth "domains" support (JCLOUDS-1414) -- File Changes