Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #25822 from edx/feanil/celery_worker_fix

Browse Source 
Only instantiate one celery app per process.
This commit is contained in:
Feanil Patel
2020-12-09 14:45:55 -05:00
committed by GitHub
parent 774210cf33 ab6bf348d4
commit 1984751472
2 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
cms/__init__.py
View File

@@ -5,7 +5,7 @@ registration and discovery can work correctly.
Import sorting is intentionally disabled in this module.
isort:skip_file
"""
import os
# We monkey patch Kombu's entrypoints listing because scanning through this
# accounts for the majority of LMS/Studio startup time for tests, and we don't
@@ -20,4 +20,5 @@ kombu.utils.entrypoints = lambda namespace: iter([])
# This will make sure the app is always imported when
# Django starts so that shared_task will use this app.
from .celery import APP as CELERY_APP
if os.environ.get('SERVICE_VARIANT', 'cms').startswith('cms'):
from .celery import APP as CELERY_APP

4
lms/__init__.py
View File

@@ -2,6 +2,7 @@
Celery needs to be loaded when the cms modules are so that task
registration and discovery can work correctly.
"""
import os
# We monkey patch Kombu's entrypoints listing because scanning through this
# accounts for the majority of LMS/Studio startup time for tests, and we don't
@@ -16,4 +17,5 @@ kombu.utils.entrypoints = lambda namespace: iter([])
# This will make sure the app is always imported when
# Django starts so that shared_task will use this app.
from .celery import APP as CELERY_APP
if os.environ.get('SERVICE_VARIANT', 'lms').startswith('lms'):
from .celery import APP as CELERY_APP