From 22f86aa1b5c85b6f4b73b8b9a5a1f4e21195ecd1 Mon Sep 17 00:00:00 2001 From: Leo Hemsted Date: Thu, 8 Mar 2018 13:35:53 +0000 Subject: [PATCH] Revert service callback worker eventlets We have seen problems with the service callback workers due to the db connection pool being exhausted. When the worker picks up the task, it makes a db query to get the notification, a query to get the callback url, and then closes the session before it makes the 3rd party request. However, even closing the session before the (potentially lengthy) web request wasn't enough - we've seen significant amounts of `sqlalchemy.exc.TimeoutError`s. This reverts commit 2dfbd93c7ed65028706e04762ea5102c60e5acbf --- manifest-delivery-base.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/manifest-delivery-base.yml b/manifest-delivery-base.yml index e141c6482..82bad9b2d 100644 --- a/manifest-delivery-base.yml +++ b/manifest-delivery-base.yml @@ -95,6 +95,6 @@ applications: NOTIFY_APP_NAME: delivery-worker-receipts - name: notify-delivery-worker-service-callbacks - command: scripts/run_app_paas.sh celery -A run_celery.notify_celery worker --loglevel=INFO -P eventlet -c 1000 -Q service-callbacks + command: scripts/run_app_paas.sh celery -A run_celery.notify_celery worker --loglevel=INFO --concurrency=11 -Q service-callbacks env: - NOTIFY_APP_NAME: delivery-worker-service-callbacks + NOTIFY_APP_NAME: delivery-worker-service-callbacks \ No newline at end of file