From e6357c91c9b06812e27a011a6ff94b4858450301 Mon Sep 17 00:00:00 2001 From: Katie Smith Date: Tue, 20 Apr 2021 15:34:49 +0100 Subject: [PATCH] Add more details to messages in send_broadcast_provider_message task This ensures that the log messages both contain broadcast_event id and broadcast_provider_message id. It also removes the broadcast_event reference since this isn't particularly useful in helping to find an event. --- app/celery/broadcast_message_tasks.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/app/celery/broadcast_message_tasks.py b/app/celery/broadcast_message_tasks.py index 7ef93db5b..9c05cf35c 100644 --- a/app/celery/broadcast_message_tasks.py +++ b/app/celery/broadcast_message_tasks.py @@ -163,8 +163,8 @@ def send_broadcast_provider_message(self, broadcast_event_id, provider): formatted_message_number = format_sequential_number(broadcast_provider_message.message_number) current_app.logger.info( - f'invoking cbc proxy to send ' - f'broadcast_event {broadcast_event.reference} ' + f'Invoking cbc proxy to send broadcast_provider_message with ID of {broadcast_provider_message.id} ' + f'and broadcast_event ID of {broadcast_event_id} ' f'msgType {broadcast_event.message_type}' ) @@ -215,8 +215,8 @@ def send_broadcast_provider_message(self, broadcast_event_id, provider): except CBCProxyRetryableException as exc: delay = get_retry_delay(self.request.retries) current_app.logger.exception( - f'Retrying send_broadcast_provider_message for broadcast_event {broadcast_event_id} and ' + - f'provider {provider} in {delay} seconds' + f'Retrying send_broadcast_provider_message for broadcast event {broadcast_event_id}, ' + f'provider message {broadcast_provider_message.id}, provider {provider} in {delay} seconds' ) self.retry(