openstack/kolla-ansible
Code Issues Proposed changes

Merge "Delete stale fluent config on restart"

Browse Source
This commit is contained in:
Zuul 2020-03-10 12:05:15 +00:00 committed by Gerrit Code Review
commit 5fbadcd8c3
2 changed files with 15 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
ansible/roles/common/templates/fluentd.json.j2
View File

@ -25,32 +25,32 @@
{# Copy all configuration files in input/ directory to include #} {# Copy all configuration files in input/ directory to include #}
{# custom input configs. #} {# custom input configs. #}
{ {
"source": "{{ container_config_directory }}/input/*.conf", "source": "{{ container_config_directory }}/input",
"dest": "{{ fluentd_dir }}/input/", "dest": "{{ fluentd_dir }}/input",
"owner": "{{ fluentd_user }}", "owner": "{{ fluentd_user }}",
"perm": "0600" "perm": "0600"
}, },
{# Copy all configuration files in filter/ directory to include #} {# Copy all configuration files in filter/ directory to include #}
{# custom filter configs. #} {# custom filter configs. #}
{ {
"source": "{{ container_config_directory }}/filter/*.conf", "source": "{{ container_config_directory }}/filter",
"dest": "{{ fluentd_dir }}/filter/", "dest": "{{ fluentd_dir }}/filter",
"owner": "{{ fluentd_user }}", "owner": "{{ fluentd_user }}",
"perm": "0600" "perm": "0600"
}, },
{# Copy all configuration files in format/ directory to include #} {# Copy all configuration files in format/ directory to include #}
{# custom format configs. #} {# custom format configs. #}
{ {
"source": "{{ container_config_directory }}/format/*.conf", "source": "{{ container_config_directory }}/format",
"dest": "{{ fluentd_dir }}/format/", "dest": "{{ fluentd_dir }}/format",
"owner": "{{ fluentd_user }}", "owner": "{{ fluentd_user }}",
"perm": "0600" "perm": "0600"
}, },
{# Copy all configuration files in output/ directory to include #} {# Copy all configuration files in output/ directory to include #}
{# custom output configs. #} {# custom output configs. #}
{ {
"source": "{{ container_config_directory }}/output/*.conf", "source": "{{ container_config_directory }}/output",
"dest": "{{ fluentd_dir }}/output/", "dest": "{{ fluentd_dir }}/output",
"owner": "{{ fluentd_user }}", "owner": "{{ fluentd_user }}",
"perm": "0600" "perm": "0600"
} }

7
releasenotes/notes/bug-1862211-1c44c4a16963baad.yaml Normal file
View File

@ -0,0 +1,7 @@
---
fixes:
- |
Fixes an issue where old fluentd configuration files would persist
in the container across restarts despite being removed from the
``node_custom_config`` directory.
`LP#1862211 <https://launchpad.net/bugs/1862211>`__