From 24f93a1f50c08cc5c3c99f72361bc98793bf039c Mon Sep 17 00:00:00 2001
From: Michal Konecny <mkonecny@redhat.com>
Date: Tue, 26 Mar 2024 10:51:05 +0100
Subject: [PATCH] [mailman3] Fix the migration task

DB migration task failed on settings.py and permissions. Let's fix that.

Signed-off-by: Michal Konecny <mkonecny@redhat.com>
---
 roles/mailman3/tasks/main.yml           | 2 +-
 roles/mailman3/templates/settings.py.j2 | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/roles/mailman3/tasks/main.yml b/roles/mailman3/tasks/main.yml
index caeef77bd3..b88fb90f52 100644
--- a/roles/mailman3/tasks/main.yml
+++ b/roles/mailman3/tasks/main.yml
@@ -207,7 +207,7 @@
   ansible.builtin.template:
     src: settings.py.j2
     dest: "{{ mailman_webui_confdir }}/settings.py"
-    owner: root
+    owner: mailman
     group: apache
     mode: 0640
   tags:
diff --git a/roles/mailman3/templates/settings.py.j2 b/roles/mailman3/templates/settings.py.j2
index 55c32d3e99..cfc9951d6d 100644
--- a/roles/mailman3/templates/settings.py.j2
+++ b/roles/mailman3/templates/settings.py.j2
@@ -336,14 +336,14 @@ LOGGING = {
     'handlers': {
         'mail_admins': {
             'level': 'ERROR',
-            'filters': ['require_debug_false', 'exclude_useless_errors'],
+            'filters': ['require_debug_false'],
             'class': 'django.utils.log.AdminEmailHandler'
         },
         'file':{
             'level': 'DEBUG',
             #'class': 'logging.handlers.RotatingFileHandler',
             'class': 'logging.handlers.WatchedFileHandler',
-            'filename': '/var/log/hyperkitty/hyperkitty.log',
+            'filename': '{{ mailman_log_directory }}/hyperkitty.log',
             'formatter': 'verbose',
         },
         'null': {