Merge "Make config provided CA optional"
This commit is contained in:
commit
e966e643db
|
@ -460,7 +460,8 @@ class HAOpenStackCharm(OpenStackAPICharm):
|
|||
return [{
|
||||
'key': self.config_defined_ssl_key.decode('utf-8'),
|
||||
'cert': self.config_defined_ssl_cert.decode('utf-8'),
|
||||
'ca': self.config_defined_ssl_ca.decode('utf-8'),
|
||||
'ca': (self.config_defined_ssl_ca.decode('utf-8')
|
||||
if self.config_defined_ssl_ca else None),
|
||||
'cn': None}]
|
||||
elif keystone_interface:
|
||||
keys_and_certs = []
|
||||
|
|
|
@ -568,6 +568,15 @@ class TestHAOpenStackCharm(BaseOpenStackCharmTest):
|
|||
self.target.get_certs_and_keys(),
|
||||
[{'key': 'key', 'cert': 'cert', 'ca': 'ca', 'cn': None}])
|
||||
|
||||
def test_get_certs_and_keys_noca(self):
|
||||
config = {
|
||||
'ssl_key': base64.b64encode(b'key'),
|
||||
'ssl_cert': base64.b64encode(b'cert')}
|
||||
self.patch_target('config', new=config)
|
||||
self.assertEqual(
|
||||
self.target.get_certs_and_keys(),
|
||||
[{'key': 'key', 'cert': 'cert', 'ca': None, 'cn': None}])
|
||||
|
||||
def test_get_certs_and_keys_ks_interface(self):
|
||||
class KSInterface(object):
|
||||
def get_ssl_key(self, key):
|
||||
|
|
Loading…
Reference in New Issue