Remove extra configuration for InfluxDB

InfluxDB config file merging isn't supported in Kolla-Ansible because
it uses 'nested sections' which aren't supported by merge_configs. If
no override file is specified, Kayobe will write out an empty config
file which will then be used as the InfluxDB config file, breaking
InfluxDB. To prevent that happening this change removes the extra
config in Kayobe. It also fixes the directory to which the 'glob'
collected config is copied to, as Kolla-Ansible doesn't look for
Influxdb config files in the influxdb folder.

Change-Id: Iee4b7987934045f0355b4a87cebaebc1aa2bbe77
Story: 2003951
Task: 26868
This commit is contained in:
Doug Szumski 2018-10-03 09:42:43 +01:00
parent f479413b5e
commit 89560dd65f
8 changed files with 3 additions and 24 deletions

View File

@ -106,7 +106,6 @@
- { name: glance, file: glance.conf } - { name: glance, file: glance.conf }
- { name: grafana, file: grafana.ini } - { name: grafana, file: grafana.ini }
- { name: heat, file: heat.conf } - { name: heat, file: heat.conf }
- { name: influxdb, file: influxdb.conf }
- { name: inspector, file: ironic-inspector.conf } - { name: inspector, file: ironic-inspector.conf }
- { name: ironic, file: ironic.conf } - { name: ironic, file: ironic.conf }
- { name: ironic_dnsmasq, file: ironic/ironic-dnsmasq.conf } - { name: ironic_dnsmasq, file: ironic/ironic-dnsmasq.conf }
@ -211,7 +210,6 @@
kolla_extra_grafana: "{{ kolla_extra_config.grafana | default }}" kolla_extra_grafana: "{{ kolla_extra_config.grafana | default }}"
kolla_extra_heat: "{{ kolla_extra_config.heat | default }}" kolla_extra_heat: "{{ kolla_extra_config.heat | default }}"
kolla_extra_inspector: "{{ kolla_extra_config.inspector | default }}" kolla_extra_inspector: "{{ kolla_extra_config.inspector | default }}"
kolla_extra_influxdb: "{{ kolla_extra_config.influxdb | default }}"
kolla_extra_ironic: "{{ kolla_extra_config.ironic | default }}" kolla_extra_ironic: "{{ kolla_extra_config.ironic | default }}"
kolla_extra_ironic_dnsmasq: "{{ kolla_extra_config.ironic_dnsmasq | default }}" kolla_extra_ironic_dnsmasq: "{{ kolla_extra_config.ironic_dnsmasq | default }}"
kolla_extra_kafka: "{{ kolla_extra_config.kafka | default }}" kolla_extra_kafka: "{{ kolla_extra_config.kafka | default }}"

View File

@ -81,9 +81,6 @@ kolla_enable_horizon:
# Whether to enable InfluxDB. # Whether to enable InfluxDB.
kolla_enable_influxdb: kolla_enable_influxdb:
# Free form extra configuration to append to influxdb.conf.
kolla_extra_influxdb:
############################################################################### ###############################################################################
# Ironic configuration. # Ironic configuration.

View File

@ -44,7 +44,6 @@ def test_service_config_directory(host, path):
'grafana', 'grafana',
'heat', 'heat',
'horizon', 'horizon',
'influxdb',
'ironic', 'ironic',
'kafka', 'kafka',
'magnum', 'magnum',

View File

@ -40,9 +40,6 @@ provisioner:
foo=bar foo=bar
kolla_enable_horizon: true kolla_enable_horizon: true
kolla_enable_influxdb: true kolla_enable_influxdb: true
kolla_extra_influxdb: |
[extra-influxdb.conf]
foo=bar
kolla_enable_ironic: true kolla_enable_ironic: true
kolla_extra_ironic: | kolla_extra_ironic: |
[extra-ironic.conf] [extra-ironic.conf]

View File

@ -36,7 +36,6 @@ testinfra_hosts = testinfra.utils.ansible_runner.AnsibleRunner(
'grafana', 'grafana',
'heat', 'heat',
'horizon', 'horizon',
'influxdb',
'ironic', 'ironic',
'kafka', 'kafka',
'keystone', 'keystone',
@ -65,7 +64,6 @@ def test_service_config_directory(host, path):
'glance.conf', 'glance.conf',
'grafana.ini', 'grafana.ini',
'heat.conf', 'heat.conf',
'influxdb.conf',
'ironic.conf', 'ironic.conf',
'ironic-inspector.conf', 'ironic-inspector.conf',
'kafka.server.properties', 'kafka.server.properties',

View File

@ -20,7 +20,6 @@
- { src: glance.conf.j2, dest: glance.conf, enabled: "{{ kolla_enable_glance }}" } - { src: glance.conf.j2, dest: glance.conf, enabled: "{{ kolla_enable_glance }}" }
- { src: grafana.ini.j2, dest: grafana.ini, enabled: "{{ kolla_enable_grafana }}" } - { src: grafana.ini.j2, dest: grafana.ini, enabled: "{{ kolla_enable_grafana }}" }
- { src: heat.conf.j2, dest: heat.conf, enabled: "{{ kolla_enable_heat }}" } - { src: heat.conf.j2, dest: heat.conf, enabled: "{{ kolla_enable_heat }}" }
- { src: influxdb.conf.j2, dest: influxdb.conf, enabled: "{{ kolla_enable_influxdb }}" }
- { src: ironic.conf.j2, dest: ironic.conf, enabled: "{{ kolla_enable_ironic }}" } - { src: ironic.conf.j2, dest: ironic.conf, enabled: "{{ kolla_enable_ironic }}" }
- { src: ironic-dnsmasq.conf.j2, dest: ironic/ironic-dnsmasq.conf, enabled: "{{ kolla_enable_ironic }}" } - { src: ironic-dnsmasq.conf.j2, dest: ironic/ironic-dnsmasq.conf, enabled: "{{ kolla_enable_ironic }}" }
- { src: ironic-inspector.conf.j2, dest: ironic-inspector.conf, enabled: "{{ kolla_enable_ironic }}" } - { src: ironic-inspector.conf.j2, dest: ironic-inspector.conf, enabled: "{{ kolla_enable_ironic }}" }

View File

@ -1,9 +0,0 @@
# {{ ansible_managed }}
{% if kolla_extra_influxdb %}
#######################
# Extra configuration
#######################
{{ kolla_extra_influxdb }}
{% endif %}

View File

@ -60,9 +60,9 @@ kolla_openstack_custom_config:
patterns: "*" patterns: "*"
enabled: "{{ kolla_enable_horizon }}" enabled: "{{ kolla_enable_horizon }}"
# InfluxDB. # InfluxDB.
- src: "{{ kolla_extra_config_path }}/influxdb" - src: "{{ kolla_extra_config_path }}/"
dest: "{{ kolla_node_custom_config_path }}/influxdb" dest: "{{ kolla_node_custom_config_path }}/"
patterns: "*" patterns: "influx*"
enabled: "{{ kolla_enable_influxdb }}" enabled: "{{ kolla_enable_influxdb }}"
# Storm. # Storm.
- src: "{{ kolla_extra_config_path }}/storm" - src: "{{ kolla_extra_config_path }}/storm"