summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatthew Edmonds <edmondsw@us.ibm.com>2015-03-31 09:53:15 -0400
committergordon chung <gord@live.ca>2015-04-20 13:46:12 -0400
commit7d7e4b878215c609653989a1ab5e9257b34ee5c6 (patch)
tree44fec13c3dd82abd811713dbded785a1a4477280
parentccf96f1a50d8844cae7520c788bf6c13f5e806f6 (diff)
downloadpython-ceilometerclient-7d7e4b878215c609653989a1ab5e9257b34ee5c6.tar.gz
ceilometerclient insecure argument no longer works
The os_insecure argument needs to be parsed and passed to the auth plugin for non-shell usage. Change-Id: Id45a680396a9b646b273d971a0f6c274e95019a6 Closes-Bug: #1438750 (cherry picked from commit a5df77d8730284e278fde087e8bb3c2520c6ddb1)
-rw-r--r--ceilometerclient/client.py4
-rw-r--r--ceilometerclient/tests/test_client.py9
2 files changed, 12 insertions, 1 deletions
diff --git a/ceilometerclient/client.py b/ceilometerclient/client.py
index 6601bec..c249a88 100644
--- a/ceilometerclient/client.py
+++ b/ceilometerclient/client.py
@@ -227,6 +227,7 @@ def _adjust_kwargs(kwargs):
'region_name': kwargs.get('os_region_name'),
'service_type': kwargs.get('os_service_type'),
'endpoint_type': kwargs.get('os_endpoint_type'),
+ 'insecure': kwargs.get('os_insecure'),
'cacert': kwargs.get('os_cacert'),
'cert_file': kwargs.get('os_cert'),
'key_file': kwargs.get('os_key'),
@@ -300,7 +301,7 @@ def get_client(version, **kwargs):
* os_auth_url: endpoint to authenticate against
* os_cert|os_cacert: path of CA TLS certificate
* os_key: SSL private key
- * insecure: allow insecure SSL (no cert verification)
+ * os_insecure: allow insecure SSL (no cert verification)
"""
endpoint = kwargs.get('os_endpoint') or kwargs.get('ceilometer_url')
@@ -313,6 +314,7 @@ def get_auth_plugin(endpoint, **kwargs):
service_type=kwargs.get('service_type'),
token=kwargs.get('token'),
endpoint_type=kwargs.get('endpoint_type'),
+ insecure=kwargs.get('insecure'),
cacert=kwargs.get('cacert'),
tenant_id=kwargs.get('project_id') or kwargs.get('tenant_id'),
endpoint=endpoint,
diff --git a/ceilometerclient/tests/test_client.py b/ceilometerclient/tests/test_client.py
index 45472a4..8297dad 100644
--- a/ceilometerclient/tests/test_client.py
+++ b/ceilometerclient/tests/test_client.py
@@ -83,6 +83,7 @@ class ClientTest(utils.BaseTestCase):
'endpoint_type': None,
'auth_url': 'http://no.where',
'tenant_id': None,
+ 'insecure': None,
'cacert': None,
'password': 'password',
'user_domain_name': 'default',
@@ -147,6 +148,14 @@ class ClientTest(utils.BaseTestCase):
self.assertEqual(('/path/to/cert', '/path/to/keycert'),
client.client.cert)
+ def test_v2_client_insecure(self):
+ env = FAKE_ENV.copy()
+ env.pop('auth_plugin')
+ env['insecure'] = 'True'
+ client = self.create_client(env)
+ self.assertIn('insecure', client.auth_plugin.opts)
+ self.assertEqual('True', client.auth_plugin.opts['insecure'])
+
class ClientTest2(ClientTest):
@staticmethod