From b62cad113ac10930f24dd3f7b946f979a4b8c76d Mon Sep 17 00:00:00 2001 From: yatin Date: Thu, 19 Jan 2017 11:14:25 +0530 Subject: [PATCH] Fix: swarm cluster-config, bay-config with tls_disabled magnum cluster-config, bay-config for swarm cluster didn't work with tls_disabled. With this patch following is done:- if tls_disabled then set DOCKER_TLS_VERIFY="" if not tls_disabled then set DOCKER_TLS_VERIFY=True Change-Id: I57eb1d49b5157958f9c5f6d89a613896300fdb6e Closes-Bug: #1647254 --- magnumclient/v1/bays_shell.py | 6 ++++++ magnumclient/v1/clusters_shell.py | 6 ++++++ 2 files changed, 12 insertions(+) diff --git a/magnumclient/v1/bays_shell.py b/magnumclient/v1/bays_shell.py index 94ae574f..a8a960b9 100644 --- a/magnumclient/v1/bays_shell.py +++ b/magnumclient/v1/bays_shell.py @@ -302,6 +302,9 @@ def _config_bay_swarm(bay, baymodel, cfg_dir, force=False): "setenv DOCKER_CERT_PATH %(cfg_dir)s\n" "setenv DOCKER_TLS_VERIFY %(tls)s\n" % {'docker_host': bay.api_address, + 'cfg_dir': cfg_dir, + 'tls': ""} if baymodel.tls_disabled else + {'docker_host': bay.api_address, 'cfg_dir': cfg_dir, 'tls': not baymodel.tls_disabled} ) @@ -310,6 +313,9 @@ def _config_bay_swarm(bay, baymodel, cfg_dir, force=False): "export DOCKER_CERT_PATH=%(cfg_dir)s\n" "export DOCKER_TLS_VERIFY=%(tls)s\n" % {'docker_host': bay.api_address, + 'cfg_dir': cfg_dir, + 'tls': ""} if baymodel.tls_disabled else + {'docker_host': bay.api_address, 'cfg_dir': cfg_dir, 'tls': not baymodel.tls_disabled} ) diff --git a/magnumclient/v1/clusters_shell.py b/magnumclient/v1/clusters_shell.py index 7eae13db..949370a8 100644 --- a/magnumclient/v1/clusters_shell.py +++ b/magnumclient/v1/clusters_shell.py @@ -297,6 +297,9 @@ def _config_cluster_swarm(cluster, cluster_template, cfg_dir='.', force=False): "setenv DOCKER_CERT_PATH %(cfg_dir)s\n" "setenv DOCKER_TLS_VERIFY %(tls)s\n" % {'docker_host': cluster.api_address, + 'cfg_dir': cfg_dir, + 'tls': ""} if cluster_template.tls_disabled else + {'docker_host': cluster.api_address, 'cfg_dir': cfg_dir, 'tls': not cluster_template.tls_disabled} ) @@ -305,6 +308,9 @@ def _config_cluster_swarm(cluster, cluster_template, cfg_dir='.', force=False): "export DOCKER_CERT_PATH=%(cfg_dir)s\n" "export DOCKER_TLS_VERIFY=%(tls)s\n" % {'docker_host': cluster.api_address, + 'cfg_dir': cfg_dir, + 'tls': ""} if cluster_template.tls_disabled else + {'docker_host': cluster.api_address, 'cfg_dir': cfg_dir, 'tls': not cluster_template.tls_disabled} )