Skip to content
This repository was archived by the owner on Aug 3, 2026. It is now read-only.

refactor: add CELERY_RESULT_BACKEND with default django-cache - #197

Merged
navinkarkera merged 1 commit into
opencraft-release/maple.3from
navin/oc-maple-set-celery-result
Jun 9, 2022
Merged

refactor: add CELERY_RESULT_BACKEND with default django-cache#197
navinkarkera merged 1 commit into
opencraft-release/maple.3from
navin/oc-maple-set-celery-result

Conversation

@navinkarkera

Copy link
Copy Markdown
Member

Configuration Pull Request

Cherry pick of openedx-unsupported#6720

Note: This PR has a corresponding edx-platform PR open-craft/openedx-platform#478

Description

At Opencraft, we want to run multiple instances of celery workers in separate servers. The celery workers need to be able to share task results. Currently, the CELERY_RESULT_BACKEND is hard coded to use django-cache.

So this PR allows to setup CELERY_RESULT_BACKEND via configuration.

To use django ORM as celery backend, one can set CELERY_RESULT_BACKEND to django-db as described in docs.

Supporting information

Deadline

"None"

@navinkarkera
navinkarkera requested a review from kaustavb12 June 8, 2022 04:32
@navinkarkera
navinkarkera force-pushed the navin/oc-maple-set-celery-result branch from b89a3a3 to 273ae73 Compare June 8, 2022 04:34
@navinkarkera navinkarkera changed the title update: add CELERY_RESULT_BACKEND with default django-cache refactor: add CELERY_RESULT_BACKEND with default django-cache Jun 8, 2022

@kaustavb12 kaustavb12 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

  • I tested this: Changes tested in PR sandbox
  • I read through the code
  • Includes documentation

@navinkarkera
navinkarkera merged commit 0f0fd18 into opencraft-release/maple.3 Jun 9, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants