Skip to content

E
ecommerce-worker
Commit 2a243efb by Renzo Lucioni
Options

Log warnings instead of errors when a task fails but can be retried

parent 3ea44c9f
Showing with 2 additions and 2 deletions
ecommerce_worker/fulfillment/v1/tasks.py
...@@ -48,9 +48,9 @@ def fulfill_order(self, order_number): ...@@ -48,9 +48,9 @@ def fulfill_order(self, order_number):
# the exception is re-raised. # the exception is re-raised.
retries = self.request.retries retries = self.request.retries
if retries == max_fulfillment_retries: if retries == max_fulfillment_retries:
logger.exception('Fulfillment of order [%s] failed.', order_number) logger.exception('Fulfillment of order [%s] failed. Giving up.', order_number)
else: else:
logger.exception('Fulfillment of order [%s] failed. Retrying.', order_number) logger.warning('Fulfillment of order [%s] failed. Retrying.', order_number)
countdown = 2 ** retries countdown = 2 ** retries
raise self.retry(exc=exc, countdown=countdown, max_retries=max_fulfillment_retries) raise self.retry(exc=exc, countdown=countdown, max_retries=max_fulfillment_retries)
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment