summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSteve Kuznetsov <skuznets@redhat.com>2017-04-13 08:56:22 -0400
committerSteve Kuznetsov <skuznets@redhat.com>2017-04-13 08:56:22 -0400
commit669c3c656d4783e07ebd3f7fa4cf6853449ff305 (patch)
tree17d76d1b198057a31df9b29d7602b0944d34934d
parent8515131a88fa3105ade6d209a84b5d6e9ef84e6d (diff)
downloadopenshift-669c3c656d4783e07ebd3f7fa4cf6853449ff305.tar.gz
openshift-669c3c656d4783e07ebd3f7fa4cf6853449ff305.tar.bz2
openshift-669c3c656d4783e07ebd3f7fa4cf6853449ff305.tar.xz
openshift-669c3c656d4783e07ebd3f7fa4cf6853449ff305.zip
Cast etcd_debug to a boolean
In the current implementation, any user-provided string in the `etcd_debug` variable will be placed into `etcd.conf`. The YAML and Ansible boolean parsing is more generous than the Golang one, so valid YAML booleans like `no` will be invalid when passed to etcd. Casting to a boolean before casting to a string normalizes the field. Signed-off-by: Steve Kuznetsov <skuznets@redhat.com>
-rw-r--r--roles/etcd/templates/etcd.conf.j22
1 files changed, 1 insertions, 1 deletions
diff --git a/roles/etcd/templates/etcd.conf.j2 b/roles/etcd/templates/etcd.conf.j2
index 9151dd0bd..1b5598f46 100644
--- a/roles/etcd/templates/etcd.conf.j2
+++ b/roles/etcd/templates/etcd.conf.j2
@@ -62,7 +62,7 @@ ETCD_PEER_KEY_FILE={{ etcd_peer_key_file }}
{% endif -%}
#[logging]
-ETCD_DEBUG="{{ etcd_debug | default(false) | string }}"
+ETCD_DEBUG="{{ etcd_debug | default(false) | bool | string }}"
{% if etcd_log_package_levels is defined %}
ETCD_LOG_PACKAGE_LEVELS="{{ etcd_log_package_levels }}"
{% endif %}