summaryrefslogtreecommitdiffstats
path: root/python-dockering
diff options
context:
space:
mode:
authorMichael Hwang <mhwang@research.att.com>2017-10-18 16:59:41 -0400
committerMichael Hwang <mhwang@research.att.com>2017-11-01 15:30:04 -0400
commitb72e025d7e04efc7babbf2988531fcb3b431dc3f (patch)
tree991161a8e9554f870729e954cd0e03f37565fc17 /python-dockering
parent310c9c1b71529a26a99f47852f5c566df8fbc909 (diff)
Turn off cert validation
Change-Id: Ie43b6ffd4b50ee08ad65015d43ec262ba0b2b7f8 Issue-Id: DCAEGEN2-189 Signed-off-by: Michael Hwang <mhwang@research.att.com>
Diffstat (limited to 'python-dockering')
-rw-r--r--python-dockering/ChangeLog.md4
-rw-r--r--python-dockering/dockering/config_building.py5
-rw-r--r--python-dockering/setup.py2
-rw-r--r--python-dockering/tests/test_config_building.py3
-rw-r--r--python-dockering/tests/test_core.py3
5 files changed, 14 insertions, 3 deletions
diff --git a/python-dockering/ChangeLog.md b/python-dockering/ChangeLog.md
index 9c7781d..b222e5d 100644
--- a/python-dockering/ChangeLog.md
+++ b/python-dockering/ChangeLog.md
@@ -5,6 +5,10 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](http://keepachangelog.com/)
and this project adheres to [Semantic Versioning](http://semver.org/).
+## [1.4.0]
+
+* Turn off cert verification for https healthchecks. See comments in the the code.
+
## [1.3.0]
* Fix issue with returning container ports as dict keys when list was expected
diff --git a/python-dockering/dockering/config_building.py b/python-dockering/dockering/config_building.py
index 564e5de..08fd1c0 100644
--- a/python-dockering/dockering/config_building.py
+++ b/python-dockering/dockering/config_building.py
@@ -60,6 +60,11 @@ def create_envs_healthcheck(docker_config, default_interval="15s",
# WATCH: HTTPS health checks don't work. Seems like Registrator bug.
# Submitted issue https://github.com/gliderlabs/registrator/issues/516
envs["SERVICE_CHECK_HTTPS"] = hc["endpoint"]
+ # Went with one of the suggestiong from the posted issue above. This
+ # author is skeptical whether https healthchecks will ever work with
+ # server cert verification because the hostname is actually the ip
+ # address.
+ envs["SERVICE_CHECK_TLS_SKIP_VERIFY"] = "true"
utils.logger.warn("Https-based health checks may not work because Registrator issue #516")
elif hc["type"] == "script":
envs["SERVICE_CHECK_SCRIPT"] = hc["script"]
diff --git a/python-dockering/setup.py b/python-dockering/setup.py
index ab87e62..45da8de 100644
--- a/python-dockering/setup.py
+++ b/python-dockering/setup.py
@@ -23,7 +23,7 @@ from setuptools import setup
setup(
name='onap-dcae-dockering',
description='Library used to manage Docker containers in DCAE',
- version="1.3.0",
+ version="1.4.0",
license='Apache 2',
author="Michael Hwang",
email="mhwang@research.att.com",
diff --git a/python-dockering/tests/test_config_building.py b/python-dockering/tests/test_config_building.py
index ff1b409..16ecb58 100644
--- a/python-dockering/tests/test_config_building.py
+++ b/python-dockering/tests/test_config_building.py
@@ -93,7 +93,8 @@ def test_create_envs_healthcheck():
expected = {
"SERVICE_CHECK_HTTPS": endpoint,
"SERVICE_CHECK_INTERVAL": interval,
- "SERVICE_CHECK_TIMEOUT": timeout
+ "SERVICE_CHECK_TIMEOUT": timeout,
+ "SERVICE_CHECK_TLS_SKIP_VERIFY": "true"
}
assert expected == doc.create_envs_healthcheck(docker_config)
diff --git a/python-dockering/tests/test_core.py b/python-dockering/tests/test_core.py
index f66f88c..76e027b 100644
--- a/python-dockering/tests/test_core.py
+++ b/python-dockering/tests/test_core.py
@@ -30,7 +30,8 @@ def test_create_client():
# Bad - Could not connect to docker engine
with pytest.raises(DockerConnectionError):
- doc.create_client("fake", 2376, reauth=True)
+ doc.create_client("fake", 2376, reauth=True, logins=[{
+ "registry": "nowhere", "username": "bob", "password": "123"}])
# TODO: Does pytest provide an env file?