New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Pin redis version since 3.0 is not compatible with Celery #3374

Merged

maarcingebala merged 1 commit into saleor:master from jxltom:fix-celery-redis

Dec 3, 2018

Contributor

jxltom commented Dec 3, 2018 •

edited

Closes #3364.

As discussed here celery/celery#5175, redis 3.0 is not supported by celery and is fixed in celery/celery#5176 in Celery.

This patch can be removed after Celery 4.2.2 is released

Pull Request Checklist

Privileged views and APIs are guarded by proper permission checks.
All visible strings are translated with proper context.
All data-formatting is locale-aware (dates, numbers, and so on).
Database queries are optimized and the number of queries is constant.
The changes are tested.
The code is documented (docstrings, project documentation).
GraphQL schema and type definitions are up to date.


          Pin redis version since 3.0 is not compatible with Celery

a47a589

maarcingebala approved these changes

View reviewed changes

maarcingebala merged commit d57dd1e into saleor:master

jxltom deleted the fix-celery-redis branch

December 3, 2018 12:09

jxltom mentioned this pull request

Fix version mismatch between celery and redis-py #3831

Merged

9 tasks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment