diff options
author | Steve Kuznetsov <skuznets@redhat.com> | 2017-04-13 08:56:22 -0400 |
---|---|---|
committer | Steve Kuznetsov <skuznets@redhat.com> | 2017-04-13 08:56:22 -0400 |
commit | 669c3c656d4783e07ebd3f7fa4cf6853449ff305 (patch) | |
tree | 17d76d1b198057a31df9b29d7602b0944d34934d | |
parent | 8515131a88fa3105ade6d209a84b5d6e9ef84e6d (diff) | |
download | openshift-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.j2 | 2 |
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 %} |