openstack/neutron
Code Issues Proposed changes

Fix NoSuchOptError on identity config option lookup

Browse Source 
Change 1afca56b059 moved a few options around and that
broke the Tempest API clients module. This patch adds
the logic to handle the change.

Closes-bug: #1614799

Change-Id: I5ef0e7cc79609ba9df31d6573fe935a6e8d837f4
This commit is contained in:
Armando Migliaccio 2016-08-18 19:24:40 -07:00
parent 94fc633124
commit af9ad0c35e
1 changed files with 18 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
neutron/tests/tempest/api/clients.py
View File

@ -13,6 +13,8 @@
# License for the specific language governing permissions and limitations # License for the specific language governing permissions and limitations
# under the License. # under the License.
from oslo_config import cfg
from tempest.lib.services.compute import keypairs_client from tempest.lib.services.compute import keypairs_client
from tempest.lib.services.compute import servers_client from tempest.lib.services.compute import servers_client
from tempest.lib.services.identity.v2 import tenants_client from tempest.lib.services.identity.v2 import tenants_client
@ -28,12 +30,22 @@ class Manager(manager.Manager):
""" """
Top level manager for OpenStack tempest clients Top level manager for OpenStack tempest clients
""" """
default_params = { try:
'disable_ssl_certificate_validation': default_params = {
CONF.identity.disable_ssl_certificate_validation, 'disable_ssl_certificate_validation':
'ca_certs': CONF.identity.ca_certificates_file, CONF.service_clients.disable_ssl_certificate_validation,
'trace_requests': CONF.debug.trace_requests 'ca_certs': CONF.service_clients.ca_certificates_file,
} 'trace_requests': CONF.debug.trace_requests
}
except cfg.NoSuchOptError:
# TODO(armax): remove this except when a new tempest release
# > 12.1.0 includes change 1afca56b05
default_params = {
'disable_ssl_certificate_validation':
CONF.identity.disable_ssl_certificate_validation,
'ca_certs': CONF.identity.ca_certificates_file,
'trace_requests': CONF.debug.trace_requests
}
# NOTE: Tempest uses timeout values of compute API if project specific # NOTE: Tempest uses timeout values of compute API if project specific
# timeout values don't exist. # timeout values don't exist.