Skip to content

Commit

Permalink
Merge pull request #1482 from GSA/change_retries2
Browse files Browse the repository at this point in the history
change retries
  • Loading branch information
ccostino authored Dec 12, 2024
2 parents 560ef3f + 6bc3290 commit b993ed9
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions app/celery/tasks.py
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,7 @@ def __total_sending_limits_for_job_exceeded(service, job, job_id):
return True


@notify_celery.task(bind=True, name="save-sms", max_retries=5, default_retry_delay=300)
@notify_celery.task(bind=True, name="save-sms", max_retries=2, default_retry_delay=600)
def save_sms(self, service_id, notification_id, encrypted_notification, sender_id=None):
"""Persist notification to db and place notification in queue to send to sns."""
notification = encryption.decrypt(encrypted_notification)
Expand Down Expand Up @@ -315,7 +315,7 @@ def save_api_email(self, encrypted_notification):


@notify_celery.task(
bind=True, name="save-api-sms", max_retries=5, default_retry_delay=300
bind=True, name="save-api-sms", max_retries=2, default_retry_delay=600
)
def save_api_sms(self, encrypted_notification):
save_api_email_or_sms(self, encrypted_notification)
Expand Down

0 comments on commit b993ed9

Please sign in to comment.