openstack/tempest
Code Issues Proposed changes

Merge "Don't read config in Manager class definition"

Browse Source
This commit is contained in:
Zuul 2017-10-19 07:17:58 +00:00 committed by Gerrit Code Review
commit 10973fb725
1 changed files with 4 additions and 2 deletions

6
tempest/clients.py

@ -24,8 +24,6 @@ CONF = config.CONF
class Manager(clients.ServiceClients): class Manager(clients.ServiceClients):
"""Top level manager for OpenStack tempest clients""" """Top level manager for OpenStack tempest clients"""
default_params = config.service_client_config()
def __init__(self, credentials, scope='project'): def __init__(self, credentials, scope='project'):
"""Initialization of Manager class. """Initialization of Manager class.
@ -46,6 +44,10 @@ class Manager(clients.ServiceClients):
self._set_object_storage_clients() self._set_object_storage_clients()
self._set_image_clients() self._set_image_clients()
self._set_network_clients() self._set_network_clients()
# TODO(andreaf) This is maintained for backward compatibility
# with plugins, but it should removed eventually, since it was
# never a stable interface and it's not useful anyways
self.default_params = config.service_client_config()
def _set_network_clients(self): def _set_network_clients(self):
self.network_agents_client = self.network.AgentsClient() self.network_agents_client = self.network.AgentsClient()