Skip to content

Commit a5cdb95

Browse files
committed
Throw an exception from Celery retry()
Without an exception here, or handling the return from `_check_concurrency_limit`, a build that hits a concurrency limit in this method will both set a retry on the task, but will also continue processing the current task like normal. This results in two identical builds, then three, then four, and so on. Untested what this does to the UI though.
1 parent 8f8d46d commit a5cdb95

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

readthedocs/projects/tasks/builds.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -283,7 +283,6 @@ def _check_concurrency_limit(self):
283283
})
284284
self.retry(
285285
exc=BuildMaxConcurrencyError,
286-
throw=False,
287286
# We want to retry this build more times
288287
max_retries=25,
289288
)

0 commit comments

Comments
 (0)