openinfra/python-tempestconf
Code Issues Proposed changes

Discover services only if there is an endpoint

Browse Source 
* tempest config-tool fails to discover services whose endpoints
  are not available by giving IndexError. This patch fixes the
  discover method by checking for services only if endpoints
  are available.

Change-Id: I51acecfadff4109867a144c62cff07fd1cc97a9d
This commit is contained in:
Chandan Kumar 2017-10-17 16:30:15 +05:30
parent bd050838fd
commit fd3baf86ef
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
config_tempest/api_discovery.py
View File

@ -235,6 +235,12 @@ def discover(auth_provider, region, object_store_discovery=True,
service_catalog = 'catalog' service_catalog = 'catalog'
public_url = 'url' public_url = 'url'
# FIXME(chandankumar): It is a workaround to filter services whose
# endpoints does not exist. Once it is merged. Let's rewrite the whole
# stuff.
auth_data[service_catalog] = [data for data in auth_data[service_catalog]
if data['endpoints']]
for entry in auth_data[service_catalog]: for entry in auth_data[service_catalog]:
name = entry['type'] name = entry['type']
services[name] = dict() services[name] = dict()