diff --git a/ansible/roles/ceph/templates/ceph-mon.json.j2 b/ansible/roles/ceph/templates/ceph-mon.json.j2 index 412e778cd5ed125699e6c40ae16bfeb6c0d809c6..d3a32b3999817b87b2769a3d1f046c8783273790 100644 --- a/ansible/roles/ceph/templates/ceph-mon.json.j2 +++ b/ansible/roles/ceph/templates/ceph-mon.json.j2 @@ -12,28 +12,28 @@ "dest": "/etc/ceph/ceph.client.admin.keyring", "owner": "ceph", "perm": "0600", - "optional": "True" + "optional": true }, { "source": "{{ container_config_directory }}/ceph.client.mon.keyring", "dest": "/etc/ceph/ceph.client.mon.keyring", "owner": "ceph", "perm": "0600", - "optional": "True" + "optional": true }, { "source": "{{ container_config_directory }}/ceph.client.radosgw.keyring", "dest": "/etc/ceph/ceph.client.radosgw.keyring", "owner": "ceph", "perm": "0600", - "optional": "True" + "optional": true }, { "source": "{{ container_config_directory }}/ceph.monmap", "dest": "/etc/ceph/ceph.monmap", "owner": "ceph", "perm": "0600", - "optional": "True" + "optional": true } ] } diff --git a/ansible/roles/ceph/templates/ceph-rgw.json.j2 b/ansible/roles/ceph/templates/ceph-rgw.json.j2 index 5a0b043f4d74d3377fcb52267e4f0178b98052ba..7cb0ccdd446d5868d2483ff050cdf53d0da5e78f 100644 --- a/ansible/roles/ceph/templates/ceph-rgw.json.j2 +++ b/ansible/roles/ceph/templates/ceph-rgw.json.j2 @@ -11,15 +11,13 @@ "source": "{{ container_config_directory }}/ceph.client.admin.keyring", "dest": "/etc/ceph/ceph.client.admin.keyring", "owner": "ceph", - "perm": "0600", - "optional": "True" + "perm": "0600" }, { "source": "{{ container_config_directory }}/ceph.client.radosgw.keyring", "dest": "/etc/ceph/ceph.client.radosgw.keyring", "owner": "ceph", - "perm": "0600", - "optional": "True" + "perm": "0600" } ] } diff --git a/ansible/roles/cinder/templates/cinder-backup.json.j2 b/ansible/roles/cinder/templates/cinder-backup.json.j2 index 2210fbb1c34f482f84cd28bbafe9f7f3b567567a..3e6c27b6ef1e088098bc5ee61582d65a96525074 100644 --- a/ansible/roles/cinder/templates/cinder-backup.json.j2 +++ b/ansible/roles/cinder/templates/cinder-backup.json.j2 @@ -12,14 +12,14 @@ "dest": "/etc/ceph/ceph.client.cinder-backup.keyring", "owner": "cinder", "perm": "0600", - "optional": "true" + "optional": {{ (not enable_ceph | bool) | string | lower }} }, { "source": "{{ container_config_directory }}/ceph.conf", "dest": "/etc/ceph/ceph.conf", "owner": "cinder", "perm": "0600", - "optional": "true" + "optional": {{ (not enable_ceph | bool) | string | lower }} } ] } diff --git a/ansible/roles/cinder/templates/cinder-volume.json.j2 b/ansible/roles/cinder/templates/cinder-volume.json.j2 index 531843b37a7ffc456c042b13aa7701195d142c39..549528410983feafcc36e30378dc53c6d32846c2 100644 --- a/ansible/roles/cinder/templates/cinder-volume.json.j2 +++ b/ansible/roles/cinder/templates/cinder-volume.json.j2 @@ -12,14 +12,14 @@ "dest": "/etc/ceph/ceph.client.cinder.keyring", "owner": "cinder", "perm": "0600", - "optional": "true" + "optional": {{ (not enable_ceph | bool) | string | lower }} }, { "source": "{{ container_config_directory }}/ceph.conf", "dest": "/etc/ceph/ceph.conf", "owner": "cinder", "perm": "0600", - "optional": "true" + "optional": {{ (not enable_ceph | bool) | string | lower }} } ] } diff --git a/ansible/roles/common/templates/heka.json.j2 b/ansible/roles/common/templates/heka.json.j2 index 857e5185b480e4b1f0826f3f27241d4d40131ffc..1087c60d0f20c6eb65dcb93b5809966d8345d1c8 100644 --- a/ansible/roles/common/templates/heka.json.j2 +++ b/ansible/roles/common/templates/heka.json.j2 @@ -7,7 +7,7 @@ "dest": "/etc/heka/heka-elasticsearch.toml", "owner": "heka", "perm": "0600", - "optional": "True" + "optional": {{ (not enable_central_logging | bool) | string | lower }} }, { "source": "{{ container_config_directory }}/heka-global.toml", @@ -51,7 +51,7 @@ "dest": "/etc/heka/heka-{{ swift_service }}.toml", "owner": "heka", "perm": "0600", - "optional": "True" + "optional": {{ (not enable_swift | bool) | string | lower }} }{% if not loop.last %},{% endif %} {% endfor %} diff --git a/ansible/roles/haproxy/templates/haproxy.json.j2 b/ansible/roles/haproxy/templates/haproxy.json.j2 index 9672e239b296a4afc410e32ab95fb098b51e6812..d838dad832d1e0fbbd76df533839b6ed3c7e5c98 100644 --- a/ansible/roles/haproxy/templates/haproxy.json.j2 +++ b/ansible/roles/haproxy/templates/haproxy.json.j2 @@ -12,7 +12,7 @@ "dest": "/etc/haproxy/haproxy.pem", "owner": "root", "perm": "0600", - "optional": "true" + "optional": {{ (not kolla_enable_tls_external | bool) | string | lower }} } ] } diff --git a/docker/base/set_configs.py b/docker/base/set_configs.py index 83b635c3090dddc2140e3d948c960e7ce7459bff..dc0a7e4a53f57e1a72f73be998db09b50616e2b9 100644 --- a/docker/base/set_configs.py +++ b/docker/base/set_configs.py @@ -58,7 +58,7 @@ def validate_source(data): if not exists: if data.get('optional'): - LOG.warning("%s does not exist, but is not required", source) + LOG.info("%s does not exist, but is not required", source) return False else: LOG.error("The source to copy does not exist: %s", source)