From 569403f4aefce14614c0c71e8bcaa5b1a72cbf14 Mon Sep 17 00:00:00 2001 From: Andy Shultz Date: Mon, 12 Dec 2022 11:17:54 -0500 Subject: [PATCH] fix: update devstack with worker for cache and proper redis https://github.com/openedx/edx-platform/pull/31261 fixed celery cache behavior when not running a worker and made sure production would keep the old cache behavior, but missed these secret alternate settings files, bring them up to date. Also fixes the cms file to have the actual broker URL. --- cms/envs/devstack_with_worker.py | 2 ++ lms/envs/devstack_with_worker.py | 1 + 2 files changed, 3 insertions(+) diff --git a/cms/envs/devstack_with_worker.py b/cms/envs/devstack_with_worker.py index ea4ba857a2..959cd4f436 100644 --- a/cms/envs/devstack_with_worker.py +++ b/cms/envs/devstack_with_worker.py @@ -18,6 +18,8 @@ from cms.envs.devstack import * # Require a separate celery worker CELERY_ALWAYS_EAGER = False +CLEAR_REQUEST_CACHE_ON_TASK_COMPLETION = True +BROKER_URL = 'redis://:password@edx.devstack.redis:6379/' # Disable transaction management because we are using a worker. Views # that request a task and wait for the result will deadlock otherwise. diff --git a/lms/envs/devstack_with_worker.py b/lms/envs/devstack_with_worker.py index 0f8da07e78..84d076445c 100644 --- a/lms/envs/devstack_with_worker.py +++ b/lms/envs/devstack_with_worker.py @@ -19,6 +19,7 @@ from lms.envs.devstack import * # Require a separate celery worker CELERY_ALWAYS_EAGER = False +CLEAR_REQUEST_CACHE_ON_TASK_COMPLETION = True BROKER_URL = 'redis://:password@edx.devstack.redis:6379/' # Disable transaction management because we are using a worker. Views # that request a task and wait for the result will deadlock otherwise.