diff options
author | hongbo tian <hongbo.tianhongbo@huawei.com> | 2017-06-28 09:09:14 +0000 |
---|---|---|
committer | Gerrit Code Review <gerrit@opnfv.org> | 2017-06-28 09:09:14 +0000 |
commit | 6ddd0a92a137906ae0dd00468c5db556dda91555 (patch) | |
tree | 7359398c40be7b43eee7e066d93bc00c18ef9dd5 /dovetail/container.py | |
parent | c3e1d8cb6bb228058a152fdcdbd8dec0d2457748 (diff) | |
parent | 1fe13eec3f4fcc0c63555364b41db06bffb9978c (diff) |
Merge "https enabled checking"
Diffstat (limited to 'dovetail/container.py')
-rw-r--r-- | dovetail/container.py | 28 |
1 files changed, 17 insertions, 11 deletions
diff --git a/dovetail/container.py b/dovetail/container.py index a1e213d8..9f624f36 100644 --- a/dovetail/container.py +++ b/dovetail/container.py @@ -173,18 +173,24 @@ class Container(object): hosts_config_path) cacert_volume = "" + https_enabled = dt_utils.check_https_enabled(cls.logger) cacert = os.getenv('OS_CACERT',) - if cacert is not None: - if not os.path.isfile(cacert): - cls.logger.error("env variable 'OS_CACERT' is set to %s" - "but the file does not exist", cacert) - return None - elif not dovetail_config['config_dir'] in cacert: - cls.logger.error("OS_CACERT file has to be put in %s, which" - "can be mount into container", - dovetail_config['config_dir']) - return None - cacert_volume = ' -v %s:%s ' % (cacert, cacert) + if https_enabled == 0: + cls.logger.info("https enabled...") + if cacert is not None: + if not os.path.isfile(cacert): + cls.logger.error("env variable 'OS_CACERT' is set to %s" + "but the file does not exist", cacert) + return None + elif not dovetail_config['config_dir'] in cacert: + cls.logger.error("credential file has to be put in %s," + "which can be mount into container", + dovetail_config['config_dir']) + return None + cacert_volume = ' -v %s:%s ' % (cacert, cacert) + else: + cls.logger.warn("https enabled, OS_CACERT not set, insecure" + "connection used or OS_CACERT missed") result_volume = ' -v %s:%s ' % (dovetail_config['result_dir'], dovetail_config[type]['result']['dir']) |