From d2623de615b81754fc125ec688e077a06c633bb1 Mon Sep 17 00:00:00 2001
From: Jennifer Richards <jennifer@staff.ietf.org>
Date: Fri, 26 Apr 2024 17:00:51 -0300
Subject: [PATCH] feat: config celery logging via Django (#7371)

* feat: config celery logger via Django

* feat: Disable celery's logging config
---
 ietf/celeryapp.py | 10 ++++++++--
 ietf/settings.py  | 14 +++++++++-----
 2 files changed, 17 insertions(+), 7 deletions(-)

diff --git a/ietf/celeryapp.py b/ietf/celeryapp.py
index b36f13463..08089d509 100644
--- a/ietf/celeryapp.py
+++ b/ietf/celeryapp.py
@@ -1,14 +1,20 @@
 import os
 import scout_apm.celery
 
-from celery import Celery
+import celery
 from scout_apm.api import Config
 
 
+# Disable celery's internal logging configuration, we set it up via Django
+@celery.signals.setup_logging.connect
+def on_setup_logging(**kwargs):
+    pass
+
+
 # Set the default Django settings module for the 'celery' program
 os.environ.setdefault('DJANGO_SETTINGS_MODULE', 'ietf.settings')
 
-app = Celery('ietf')
+app = celery.Celery('ietf')
 
 # Using a string here means the worker doesn't have to serialize
 # the configuration object to child processes.
diff --git a/ietf/settings.py b/ietf/settings.py
index 990d8bf05..9f12e0b4d 100644
--- a/ietf/settings.py
+++ b/ietf/settings.py
@@ -252,17 +252,21 @@ LOGGING = {
             'level': 'INFO',
         },
         'django.security': {
-	        'handlers': ['console', ],
+            'handlers': ['console', ],
             'level': 'INFO',
         },
- 	    'oidc_provider': {
-	        'handlers': ['debug_console', ],
-	        'level': 'DEBUG',
-	    },
+        'oidc_provider': {
+            'handlers': ['debug_console', ],
+            'level': 'DEBUG',
+        },
         'datatracker': {
             'handlers': ['console', ],
             'level': 'INFO',
         },
+        'celery': {
+            'handlers': ['console'],
+            'level': 'INFO',
+        }
     },
     #
     # No logger filters