Skip to content

Commit

Permalink
Remove django-celery-beat and use the default scheduler (#3959)
Browse files Browse the repository at this point in the history
There are some issues with timezone for the `DatabaseScheduler`, so we
fallback to the default one (celery.beat.PersistentScheduler) which is
simpler --doesn't allow us to change the scheduled task from the
admin, but we don't need that.

In case we need this again, we can come back when those issues are
already fixed and revert this commit.
  • Loading branch information
humitos authored and agjohnson committed Apr 17, 2018
1 parent d458592 commit 907bad3
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 3 deletions.
2 changes: 0 additions & 2 deletions readthedocs/settings/base.py
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,6 @@ def INSTALLED_APPS(self): # noqa
'annoying',
'django_extensions',
'messages_extends',
'django_celery_beat',

# daniellindsleyrocksdahouse
'haystack',
Expand Down Expand Up @@ -245,7 +244,6 @@ def USE_PROMOS(self): # noqa
CELERY_CREATE_MISSING_QUEUES = True

CELERY_DEFAULT_QUEUE = 'celery'
CELERYBEAT_SCHEDULER = 'django_celery_beat.schedulers:DatabaseScheduler'
CELERYBEAT_SCHEDULE = {
# Ran every hour on minute 30
'hourly-remove-orphan-symlinks': {
Expand Down
1 change: 0 additions & 1 deletion requirements/pip.txt
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,6 @@ defusedxml==0.5.0
# Basic tools
redis==2.10.6
celery==4.1.0
django-celery-beat==1.1.1

# django-allauth 0.33.0 dropped support for Django 1.9
# https://django-allauth.readthedocs.io/en/latest/release-notes.html#backwards-incompatible-changes
Expand Down

0 comments on commit 907bad3

Please sign in to comment.