From 69497795448d0f13c83ebab977a3dfa3b6725eca Mon Sep 17 00:00:00 2001 From: Rebecca Law Date: Mon, 10 Apr 2017 17:13:56 +0100 Subject: [PATCH] Refactor s3upload s3upload function has been moved to notifications-utils. https://github.com/alphagov/notifications-utils/pull/138 must be merge first. Remove scratch code that I was using to test celery.send_task method Revert the update to default_retry_delay. Increase the countdown, wait 1 minute before executing the task. Some of the changes in my previous commit are included here, that branch will be merged before this one, so it will make sense. Update version of notification_utils Update the version of notifications-utils --- app/celery/tasks.py | 2 +- requirements.txt | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/app/celery/tasks.py b/app/celery/tasks.py index 070989fd0..a0458e38e 100644 --- a/app/celery/tasks.py +++ b/app/celery/tasks.py @@ -271,7 +271,7 @@ def persist_letter( handle_exception(self, notification, notification_id, e) -@notify_celery.task(bind=True, name="build-dvla-file", countdown=30, max_retries=15, default_retry_delay=30) +@notify_celery.task(bind=True, name="build-dvla-file", countdown=60, max_retries=15, default_retry_delay=300) @statsd(namespace="tasks") def build_dvla_file(self, job_id): try: diff --git a/requirements.txt b/requirements.txt index 537e3d182..0557ed9dc 100644 --- a/requirements.txt +++ b/requirements.txt @@ -31,5 +31,4 @@ awscli-cwlogs>=1.4,<1.5 git+https://github.com/alphagov/notifications-utils.git@15.0.4#egg=notifications-utils==15.0.4 - git+https://github.com/alphagov/boto.git@2.43.0-patch3#egg=boto==2.43.0-patch3