Merge "fluentd: log to a file instead of stdout"

ansible/roles/common/tasks/config.yml

@@ -297,6 +297,7 @@
     - { name: "designate", enabled: "{{ enable_designate }}" }
     - { name: "elasticsearch", enabled: "{{ enable_elasticsearch }}" }
     - { name: "etcd", enabled: "{{ enable_etcd }}" }
+    - { name: "fluentd", enabled: "yes" }
     - { name: "freezer", enabled: "{{ enable_freezer }}" }
     - { name: "glance", enabled: "{{ enable_glance }}" }
     - { name: "glance-tls-proxy", enabled: "{{ enable_glance }}" }

ansible/roles/common/templates/cron-logrotate-fluentd.conf.j2

@@ -0,0 +1,3 @@
+"/var/log/kolla/fluentd/*.log"
+{
+}

ansible/roles/common/templates/fluentd.json.j2

@@ -31,7 +31,7 @@
 {%- endmacro -%}
 
 {
-    "command": "{{ fluentd_cmd }}",
+    "command": "{{ fluentd_cmd }} -o /var/log/kolla/fluentd/fluentd.log",
     "config_files": [
         {
             "source": "{{ container_config_directory }}/td-agent.conf",
@@ -46,6 +46,11 @@
         {{ config_directory("output") }}
     ],
     "permissions": [
+        {
+            "path": "/var/log/kolla/fluentd",
+            "owner": "{{ fluentd_user }}:{{ fluentd_user }}",
+            "recurse": true
+        },
         {
             "path": "/var/log/kolla/haproxy",
             "owner": "{{ fluentd_user }}:{{ fluentd_user }}",

releasenotes/notes/bug-1888852-8735ee29f69f77b5.yaml

@@ -0,0 +1,5 @@
+---
+fixes:
+  - |
+    ``fluentd`` is now logging to ``/var/log/kolla/fluentd/fluentd.log``
+    instead of ``stdout``. `LP#1888852 <https://launchpad.net/bugs/1888852>`__