darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2025-12-25 09:51:42 -05:00
Code Issues Packages Projects Releases Wiki Activity

update kombu version to prevent 100% CPU usage

Browse Source 
This pulls in the most recent version of celery/kombu/master - it
prevents situations where the celery MainProcess would consume 100%
CPU while not distributing any tasks to the workers.

Note: If, locally, you already have kombu==4.0.2, you'll have to run
pip install --upgrade https://github.com/celery/kombu/zipball/b2f21289
to get the new version.
This commit is contained in:
Leo Hemsted
2017-07-17 14:00:55 +01:00
parent 6d1ec3eb44
commit bfdf9bc9a0
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
requirements.txt
View File

@@ -33,8 +33,9 @@ awscli-cwlogs>=1.4,<1.5
git+https://github.com/alphagov/notifications-utils.git@17.5.3#egg=notifications-utils==17.5.3
# Kombu is a library that celery uses under the hood.
# Kombu is pinned to a specific commit that brings in fixes for SQS - see https://github.com/celery/kombu/pull/693
# Kombu v4.0.2 (which ships with celery v4.0.2) doesn't work with SQS due to problems with their use of boto2, so
# Kombu migrated to boto3 - We're waiting for that to get a release version, and then to get a new version of celery
# that pulls that in. Until that point, we should override the kombu version to get these SQS fixes.
git+https://github.com/celery/kombu@09bd23bbd83344b09cbf38b7257107e560db9f25
# Additionally, kombu master also includes a fix for the main process taking 100% CPU and not distributing tasks (!)
# See https://github.com/celery/kombu/pull/693 and https://github.com/celery/kombu/pull/760
https://github.com/celery/kombu/zipball/b2f21289284496efd89acea003ff9c24105b970e