summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTomas Sedovic <tomas@sedovic.cz>2017-11-02 10:52:30 +0100
committerTomas Sedovic <tomas@sedovic.cz>2017-11-07 14:35:47 +1100
commitad84935b5021da5ab0d21ffdf630079c1a59083d (patch)
treef743a8267b1e4e299b61efc4aa81feaa27d6cde7
parent4fd33e96eed4d1d5eaca0af8f2ef3e81fcaf5498 (diff)
downloadopenshift-ad84935b5021da5ab0d21ffdf630079c1a59083d.tar.gz
openshift-ad84935b5021da5ab0d21ffdf630079c1a59083d.tar.bz2
openshift-ad84935b5021da5ab0d21ffdf630079c1a59083d.tar.xz
openshift-ad84935b5021da5ab0d21ffdf630079c1a59083d.zip
Use `null` instead of `False` where it makes sense
The `openstack_*_network_name` vars are strings, not booleans, so the absense shouldn't really be marked by `False`.
-rw-r--r--roles/openshift_openstack/defaults/main.yml4
1 files changed, 2 insertions, 2 deletions
diff --git a/roles/openshift_openstack/defaults/main.yml b/roles/openshift_openstack/defaults/main.yml
index aa03c088e..1f9c09c96 100644
--- a/roles/openshift_openstack/defaults/main.yml
+++ b/roles/openshift_openstack/defaults/main.yml
@@ -75,8 +75,8 @@ openstack_node_image: "{{ openstack_default_image_name }}"
openstack_lb_image: "{{ openstack_default_image_name }}"
openstack_etcd_image: "{{ openstack_default_image_name }}"
openstack_dns_image: "{{ openstack_default_image_name }}"
-openstack_provider_network_name: False
-openstack_external_network_name: False
+openstack_provider_network_name: null
+openstack_external_network_name: null
openstack_private_network: >-
{% if openstack_provider_network_name | default(None) -%}
{{ openstack_provider_network_name }}