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

[BB-6123] cherry-pick of celery upgrade configurations - #198

Merged
Agrendalath merged 2 commits into
opencraft-release/maple.3from
navin/backport-celery-config-maple
Jun 13, 2022
Merged

[BB-6123] cherry-pick of celery upgrade configurations#198
Agrendalath merged 2 commits into
opencraft-release/maple.3from
navin/backport-celery-config-maple

Conversation

@navinkarkera

@navinkarkera navinkarkera commented Jun 8, 2022

Copy link
Copy Markdown
Member

Description

Cherry-pick of

Dependencies

Related to open-craft/openedx-platform#479, which cherry picks celery upgrade to maple branch in opencraft.

Related tickets

…upported#6586)

* feat: make SINGLE_BEAT_IDENTIFIER configurable

* fix: celery 5.0+ does not support --app as a worker flag

* refactor: remove duplicated app declaration

* fix: --config flag is global since Celery 5.0+

* fix: replace single-beat with an updated version

* fix: add missing comma

* fix: replace maxtasksperchild worker flag

* refactor: address PR review comments

Signed-off-by: Gabor Boros <gabor.brs@gmail.com>

* refactor: remove celery<5 adjustments

Signed-off-by: Gabor Boros <gabor.brs@gmail.com>

* fix: set single-beat autorestart

Signed-off-by: Gabor Boros <gabor.brs@gmail.com>
@navinkarkera
navinkarkera force-pushed the navin/backport-celery-config-maple branch from d1bf0f1 to 6542bd4 Compare June 8, 2022 06:09
@navinkarkera navinkarkera changed the title [BB-6123] cherry-pick configuration/pull/6586 [BB-6123] cherry-pick of celery upgrade configurations Jun 8, 2022

@Agrendalath Agrendalath 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: checked Celery tasks on the sandbox
  • I read through the code
  • I checked for accessibility issues: n/a
  • Includes documentation: n/a
  • I made sure any change in configuration variables is reflected in the corresponding client's configuration-secure repository: n/a

@Agrendalath
Agrendalath merged commit 4ed5b98 into opencraft-release/maple.3 Jun 13, 2022
@Agrendalath
Agrendalath deleted the navin/backport-celery-config-maple branch June 13, 2022 10:17
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.

3 participants