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

fix: add CELERY_RESULT_BACKEND with default django-cache - #191

Merged
navinkarkera merged 2 commits into
opencraft-release/lilac.2from
navin/set-celery-result-backend-lilac.2
May 26, 2022
Merged

fix: add CELERY_RESULT_BACKEND with default django-cache#191
navinkarkera merged 2 commits into
opencraft-release/lilac.2from
navin/set-celery-result-backend-lilac.2

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#467

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"

@kaustavb12

Copy link
Copy Markdown
Member

👍

Looks good to go from my end.

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

@navinkarkera
navinkarkera merged commit 66325b0 into opencraft-release/lilac.2 May 26, 2022
farhaanbukhsh pushed a commit that referenced this pull request Jun 16, 2022
* fix: add CELERY_RESULT_BACKEND with default django-cache

* Update CHANGELOG.md
0x29a pushed a commit that referenced this pull request Jun 17, 2022
* fix: add CELERY_RESULT_BACKEND with default django-cache

* Update CHANGELOG.md
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