diff options
author | Erik Olof Gunnar Andersson <eandersson@blizzard.com> | 2017-03-09 15:49:49 -0800 |
---|---|---|
committer | Erik Olof Gunnar Andersson <eandersson@blizzard.com> | 2017-05-18 11:29:51 -0700 |
commit | 2da1a2485b336ac83331b9dd465dde744665ecba (patch) | |
tree | c50dea4780451ab01164cca3bd1d9c33d34b496a | |
parent | 60e35d16b5d944d708f781271d63ce17df31df4e (diff) | |
download | trove-2da1a2485b336ac83331b9dd465dde744665ecba.tar.gz |
Fixing PROXY_AUTH_URL not being populated properly
In a recent patch some imports were added to the top of
cmd files, before the call to load the configuration
was issued. This caused anything initilized before the
configuration to fallback on default values.
Because of this, for example PROXY_AUTH_URL found in
trove/common/remote.py would always be set to the
default value: "http://0.0.0.0:5000/v2.0".
> PROXY_AUTH_URL = CONF.trove_auth_url
This patch changes the config values in the
trove/common/*remote.py files to be lazy loaded.
Co-Authored-By: Riccardo Pittau <rpittau@blizzard.com>
Change-Id: Ie3893680c406d92fe2b20e892c29fce675d7f224
Closes-Bug: #1671655
(cherry picked from commit db320b80a568f25f59da233853c530e647a0c496)
-rw-r--r-- | trove/common/remote.py | 9 | ||||
-rw-r--r-- | trove/common/single_tenant_remote.py | 8 | ||||
-rw-r--r-- | trove/common/trove_remote.py | 4 |
3 files changed, 7 insertions, 14 deletions
diff --git a/trove/common/remote.py b/trove/common/remote.py index 0818b6a7..8b1826e2 100644 --- a/trove/common/remote.py +++ b/trove/common/remote.py @@ -27,9 +27,6 @@ from swiftclient.client import Connection CONF = cfg.CONF -PROXY_AUTH_URL = CONF.trove_auth_url -USE_SNET = CONF.backup_use_snet - def normalize_url(url): """Adds trailing slash if necessary.""" @@ -103,7 +100,7 @@ def nova_client(context, region_name=None): bypass_url=url, tenant_id=context.tenant, project_domain_name=context.project_domain_name, - auth_url=PROXY_AUTH_URL, + auth_url=CONF.trove_auth_url, auth_token=context.auth_token) client.client.auth_token = context.auth_token client.client.management_url = url @@ -132,7 +129,7 @@ def cinder_client(context, region_name=None): client = CinderClient.Client(context.user, context.auth_token, project_id=context.tenant, - auth_url=PROXY_AUTH_URL) + auth_url=CONF.trove_auth_url) client.client.auth_token = context.auth_token client.client.management_url = url return client @@ -169,7 +166,7 @@ def swift_client(context, region_name=None): client = Connection(preauthurl=url, preauthtoken=context.auth_token, tenant_name=context.tenant, - snet=USE_SNET) + snet=CONF.backup_use_snet) return client diff --git a/trove/common/single_tenant_remote.py b/trove/common/single_tenant_remote.py index 87f93756..28b3e74c 100644 --- a/trove/common/single_tenant_remote.py +++ b/trove/common/single_tenant_remote.py @@ -49,8 +49,6 @@ remote_neutron_client = \ """ -PROXY_AUTH_URL = CONF.trove_auth_url - def nova_client_trove_admin(context, region_name=None, compute_url=None): """ @@ -66,7 +64,7 @@ def nova_client_trove_admin(context, region_name=None, compute_url=None): client = NovaClient(CONF.nova_proxy_admin_user, CONF.nova_proxy_admin_pass, CONF.nova_proxy_admin_tenant_name, - auth_url=PROXY_AUTH_URL, + auth_url=CONF.trove_auth_url, service_type=CONF.nova_compute_service_type, region_name=region_name or CONF.os_region_name) @@ -88,7 +86,7 @@ def cinder_client_trove_admin(context=None): client = CinderClient.Client(CONF.nova_proxy_admin_user, CONF.nova_proxy_admin_pass, project_id=CONF.nova_proxy_admin_tenant_name, - auth_url=PROXY_AUTH_URL, + auth_url=CONF.trove_auth_url, service_type=CONF.cinder_service_type, region_name=CONF.os_region_name) @@ -110,7 +108,7 @@ def neutron_client_trove_admin(context=None): username=CONF.nova_proxy_admin_user, password=CONF.nova_proxy_admin_pass, tenant_name=CONF.nova_proxy_admin_tenant_name, - auth_url=PROXY_AUTH_URL, + auth_url=CONF.trove_auth_url, service_type=CONF.neutron_service_type, region_name=CONF.os_region_name) diff --git a/trove/common/trove_remote.py b/trove/common/trove_remote.py index 01b4141e..f6904589 100644 --- a/trove/common/trove_remote.py +++ b/trove/common/trove_remote.py @@ -23,8 +23,6 @@ from troveclient.v1 import client as TroveClient CONF = cfg.CONF -PROXY_AUTH_URL = CONF.trove_auth_url - """ NOTE(mwj, Apr 2016): @@ -47,7 +45,7 @@ def trove_client(context, region_name=None): client = TroveClient.Client(context.user, context.auth_token, project_id=context.tenant, - auth_url=PROXY_AUTH_URL) + auth_url=CONF.trove_auth_url) client.client.auth_token = context.auth_token client.client.management_url = url return client |