From 09888f74793b6c90270e5d9a9c7325f1d9189879 Mon Sep 17 00:00:00 2001 From: Leo Hemsted Date: Mon, 3 Jun 2019 11:46:07 +0100 Subject: [PATCH] ensure cronitor decorator is inside the notify_task wrapper the celery decorator should always be on the outside so that all other decorators will be captured within the celery task. We had problems with cronitor not reporting, and only for this task. --- app/celery/letters_pdf_tasks.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/celery/letters_pdf_tasks.py b/app/celery/letters_pdf_tasks.py index c83619e6c..b164e64f8 100644 --- a/app/celery/letters_pdf_tasks.py +++ b/app/celery/letters_pdf_tasks.py @@ -111,8 +111,8 @@ def get_letters_pdf(template, contact_block, filename, values): return resp.content, billable_units -@cronitor("collate-letter-pdfs-for-day") @notify_celery.task(name='collate-letter-pdfs-for-day') +@cronitor("collate-letter-pdfs-for-day") def collate_letter_pdfs_for_day(date=None): if not date: # Using the truncated date is ok because UTC to BST does not make a difference to the date,