launchpad-reviewers team mailing list archive

[İlker Emre Koç <mp+492996@xxxxxxxxxxxxxxxxxx>]

The proposal to merge ~ilkeremrekoc/launchpad:upgrade-celery into launchpad:master has been updated.

Description changed to:

Celery 5 breaks the previous CLI option distribution, we updated them to
fit with the new standard. Most importantly, options like `config`, `broker`
etc. are no longer supported to be after the `worker` and `beat` commands,
requiring to be after the `celery` command directly. We also now need to feed
the celery app directly to the commands through `-A` since newer versions of
celery no longer automatically scan for such applications.

Celery 5 also gets rid of celery.task module, which was used in our
tests, we replace that.

We also add both the dependencies and transitive dependencies of celery
5 and upgrade lazr.jobrunner since the celery version range of the
previous version was blocking us.

For more details, see:
https://code.launchpad.net/~ilkeremrekoc/launchpad/+git/launchpad/+merge/492996
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~ilkeremrekoc/launchpad:upgrade-celery into launchpad:master.