From 1af5e2abda8aa2c606e418a41659626b930c272a Mon Sep 17 00:00:00 2001 From: Eric Fischer Date: Fri, 22 Jul 2016 14:19:03 -0400 Subject: [PATCH] Retry-while-retrying is not an error in bulk email (#13068) This code will be hit if, during a retry of the send_email task, an error is encountered that means we need to retry the task again. Since the task will be retried with timed backoff up to predefinied limits, this is an expected situation and does not need to be logged as an exception. --- lms/djangoapps/bulk_email/tasks.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/lms/djangoapps/bulk_email/tasks.py b/lms/djangoapps/bulk_email/tasks.py index e3c94fd411..86b3978078 100644 --- a/lms/djangoapps/bulk_email/tasks.py +++ b/lms/djangoapps/bulk_email/tasks.py @@ -795,8 +795,11 @@ def _submit_for_retry(entry_id, email_id, to_list, global_email_context, raise retry_task except RetryTaskError as retry_error: # If the retry call is successful, update with the current progress: - log.exception(u'Task %s: email with id %d caused send_course_email task to retry.', - task_id, email_id) + log.info( + u'Task %s: email with id %d caused send_course_email task to retry again.', + task_id, + email_id + ) return subtask_status, retry_error except Exception as retry_exc: # pylint: disable=broad-except # If there are no more retries, because the maximum has been reached,