Skip to content
This repository has been archived by the owner on Nov 30, 2021. It is now read-only.

Updated to Celery 3.1.6. #350

Merged
merged 1 commit into from Dec 3, 2013
Merged

Updated to Celery 3.1.6. #350

merged 1 commit into from Dec 3, 2013

Conversation

mboersma
Copy link
Member

@mboersma mboersma commented Dec 1, 2013

We have been pinned at an old version of celery (3.0.22) due to a bug that was later fixed. The djcelery app for integrating with Django and celeryd are both deprecated. Celery integrates directly with Django now in essentially the same way it would for any generic project.

This removes a django app and several unused database tables (djcelery), moves the celery CONCURRENCY setting to settings.py (or local_settings.py). There is one post-3.1.5 celery bugfix we need, so we must pull from a git SHA until they package a newer celery.

@ghost ghost assigned gabrtv Dec 1, 2013
@mboersma
Copy link
Member Author

mboersma commented Dec 1, 2013

Please merge in conjunction with opdemand/deis-cookbook#38.

@coveralls
Copy link

Coverage Status

Coverage decreased (-48.05%) when pulling ae7f759 on update-celery into 07bdd2a on master.

@mboersma
Copy link
Member Author

mboersma commented Dec 1, 2013

Seems to have cratered our coverage report somehow. Investigating...

@gabrtv
Copy link
Member

gabrtv commented Dec 2, 2013

@mboersma we need to fix the test runner before we can merge this.

@ghost ghost assigned mboersma Dec 2, 2013
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 463dfcf on update-celery into 4e2083f on master.

@mboersma
Copy link
Member Author

mboersma commented Dec 2, 2013

Fixed the tests by adding CELERY_ALWAYS_EAGER=True to appropriate test cases, which gives the same effect as the old djcelery test loader.

gabrtv pushed a commit that referenced this pull request Dec 3, 2013
@gabrtv gabrtv merged commit 86301ea into master Dec 3, 2013
@gabrtv gabrtv deleted the update-celery branch December 3, 2013 17:21
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.

None yet

3 participants