Skip to content

Commit

Permalink
fix(usage): raise time limit and lower frequency (#2672)
Browse files Browse the repository at this point in the history
fix(usage): raise time limit and lower frequency

As discussed with @majamassarini in the DMs, there are numerous occurencies of the hourly usage statistics caching not meeting the Celery deadline. Therefore adjusting both timeout and frequency.
• Timeout: 1800s → 3600s
30min → 60min
To make sure it doesn't spam Sentry with unnecessary exceptions.
• Frequency: 3600s → 10800s
1hr →    3hr
To avoid parallel runs of the same task and also to avoid congestion
with other tasks (e.g., regular jobs).

Reviewed-by: Maja Massarini
Reviewed-by: Laura Barcziová
  • Loading branch information
2 parents 80e302f + 99bc73d commit 8bdf681
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions packit_service/celery_config.py
Original file line number Diff line number Diff line change
Expand Up @@ -51,8 +51,8 @@
},
"get_usage_statistics": {
"task": "packit_service.worker.tasks.get_usage_statistics",
"schedule": 3600.0,
"options": {"queue": "long-running", "time_limit": 1800},
"schedule": 10800.0,
"options": {"queue": "long-running", "time_limit": 3600},
},
}

Expand Down

0 comments on commit 8bdf681

Please sign in to comment.