Skip to content

Commit

Permalink
Merge pull request #1483 from GSA/main
Browse files Browse the repository at this point in the history
Production Deploy 12/12/24 - small retry hotfix
  • Loading branch information
terrazoon authored Dec 12, 2024
2 parents 00f39e5 + b993ed9 commit 42536d5
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 42536d5

Please sign in to comment.