Merge pull request #1472 from GSA/change_retry_schedule

change worker_max_tasks_per_child to 2000
GSA · Dec 10, 2024 · 0e5b08c · 0e5b08c
2 parents 2a93275 + bdb73e9
commit 0e5b08c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/clients/cloudwatch/aws_cloudwatch.py b/app/clients/cloudwatch/aws_cloudwatch.py
@@ -158,7 +158,7 @@ def check_sms(self, message_id, notification_id, created_at):
                 message["delivery"].get("phoneCarrier", "Unknown Carrier"),
             )
 
-        if time_now > (created_at + timedelta(hours=3)):
+        if time_now > (created_at + timedelta(hours=73)):
             # see app/models.py Notification. This message corresponds to "permanent-failure",
             # but we are copy/pasting here to avoid circular imports.
             return "failure", "Unable to find carrier response."

diff --git a/app/config.py b/app/config.py
@@ -167,7 +167,7 @@ class Config(object):
     current_minute = (datetime.now().minute + 1) % 60
 
     CELERY = {
-        "worker_max_tasks_per_child": 500,
+        "worker_max_tasks_per_child": 2000,
         "broker_url": REDIS_URL,
         "broker_transport_options": {
             "visibility_timeout": 310,