openstack/rally
Code Issues Proposed changes

Change the default value of use_public_urls

Browse Source 
Change the default value of the use_public_urls configuration option of
the ExistingCloud deployment engine to True.

Change-Id: I8ccb19fdef7911c2e27145758014e942086bf954
This commit is contained in:
Tzanetos Balitsaris
2014-07-04 18:24:31 +03:00
parent b5028c07d2
commit b71e389a51
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
rally/deploy/engines/existing.py
View File

@@ -31,7 +31,7 @@ class ExistingCloud(engine.EngineFactory):
"password": "password", "password": "password",
"tenant_name": "demo", "tenant_name": "demo",
"region_name": "RegionOne", "region_name": "RegionOne",
"use_public_urls": False, "use_public_urls": True,
"keystone_admin_port": 35357 "keystone_admin_port": 35357
} }
} }
@@ -73,7 +73,7 @@ class ExistingCloud(engine.EngineFactory):
consts.EndpointPermission.ADMIN, consts.EndpointPermission.ADMIN,
endpoint_dict.get('region_name'), endpoint_dict.get('region_name'),
endpoint_dict.get('use_public_urls', endpoint_dict.get('use_public_urls',
False), True),
endpoint_dict.get('admin_port', endpoint_dict.get('admin_port',
35357)) 35357))
return [admin_endpoint] return [admin_endpoint]

2
tests/deploy/engines/test_existing.py
View File

@@ -33,7 +33,7 @@ class TestExistingCloud(test.TestCase):
'password': 'myadminpass', 'password': 'myadminpass',
'tenant_name': 'demo', 'tenant_name': 'demo',
'region_name': 'RegionOne', 'region_name': 'RegionOne',
'use_public_urls': False, 'use_public_urls': True,
'admin_port': 35357 'admin_port': 35357
}, },
}, },