trozet / tacker

tacker: specifications a servicevm/device life cycle manager. See https://wiki.openstack.org/wiki/ServiceVM
http://openstack.org
Apache License 2.0
3 stars 7 forks source link

Add keystone v3 compatibility to trozet/tacker #27

Closed georgepar closed 7 years ago

georgepar commented 7 years ago

Uplift to keystoneclient v3

Replace old v2.0 specific keystone related code with v3

Signed-off-by: George Paraskevopoulos geopar@intracom-telecom.com

Add keystoneclient v3 compatibility

As per the comments of trozet instead of uplifting we keep keystoneclient v2.0 as the default and use keystoneclient v3 only when OS_AUTH_URL ends with /v3

Signed-off-by: George Paraskevopoulos geopar@intracom-telecom.com

Match auth_uri trailing version correctly

Use a regex to match the auth_uri trailing version to ignore trailing slashes.

Signed-off-by: George Paraskevopoulos geopar@intracom-telecom.com

Fix url version matching logic.

The version matching condition was inverted. This is fixed and all the version matching logic was moved in tacker.common.utils

Signed-off-by: George Paraskevopoulos geopar@intracom-telecom.com

Move version formatting check into util function

Signed-off-by: George Paraskevopoulos geopar@intracom-telecom.com

georgepar commented 7 years ago

@trozet here it is. Merge?