From 6c9d1f9b55be6a92c6fd8c54b6b28348d624fd24 Mon Sep 17 00:00:00 2001
From: Michal Arbet <michal.arbet@ultimum.io>
Date: Mon, 31 Oct 2022 11:14:48 +0100
Subject: [PATCH] Fix missing logrotate configuration for proxysql logs
This trivial patch is just adding missed logrotate
configuration for proxysql.
Closes-Bug: #1995248
Change-Id: I3ad88d03836930160b6db43a7cad63b34ffc62b0
---
ansible/roles/common/tasks/config.yml | 1 +
.../roles/common/templates/cron-logrotate-proxysql.conf.j2 | 3 +++
releasenotes/notes/bug-1995248-f454ec0198bf7c36.yaml | 5 +++++
3 files changed, 9 insertions(+)
create mode 100644 ansible/roles/common/templates/cron-logrotate-proxysql.conf.j2
create mode 100644 releasenotes/notes/bug-1995248-f454ec0198bf7c36.yaml
diff --git a/ansible/roles/common/tasks/config.yml b/ansible/roles/common/tasks/config.yml
index dd0790e25a..2bc16d4e16 100644
--- a/ansible/roles/common/tasks/config.yml
+++ b/ansible/roles/common/tasks/config.yml
@@ -194,6 +194,7 @@
- { name: "outward-rabbitmq", enabled: "{{ enable_outward_rabbitmq | bool }}" }
- { name: "placement", enabled: "{{ enable_placement | bool }}" }
- { name: "prometheus", enabled: "{{ enable_prometheus | bool }}" }
+ - { name: "proxysql", enabled: "{{ enable_proxysql | bool }}" }
- { name: "rabbitmq", enabled: "{{ enable_rabbitmq | bool }}" }
- { name: "sahara", enabled: "{{ enable_sahara | bool }}" }
- { name: "senlin", enabled: "{{ enable_senlin | bool }}" }
diff --git a/ansible/roles/common/templates/cron-logrotate-proxysql.conf.j2 b/ansible/roles/common/templates/cron-logrotate-proxysql.conf.j2
new file mode 100644
index 0000000000..7055cce4ea
--- /dev/null
+++ b/ansible/roles/common/templates/cron-logrotate-proxysql.conf.j2
@@ -0,0 +1,3 @@
+"/var/log/kolla/proxysql/*.log"
+{
+}
diff --git a/releasenotes/notes/bug-1995248-f454ec0198bf7c36.yaml b/releasenotes/notes/bug-1995248-f454ec0198bf7c36.yaml
new file mode 100644
index 0000000000..7029e6f6bd
--- /dev/null
+++ b/releasenotes/notes/bug-1995248-f454ec0198bf7c36.yaml
@@ -0,0 +1,5 @@
+---
+fixes:
+ - |
+ Fixes missing logrotate configuration for proxysql logs.
+ `LP#1995248 <https://launchpad.net/bugs/1995248>`__