Partial revert of #12369 for celery visibility timeout

agjohnson · agjohnson · commit 5962485ebef8 · 2025-08-26T12:57:58.000-07:00
diff --git a/readthedocs/settings/base.py b/readthedocs/settings/base.py
@@ -640,7 +640,7 @@ def BUILD_MEMORY_LIMIT(self):
     # https://github.com/readthedocs/readthedocs.org/issues/12317#issuecomment-3070950434
     # https://docs.celeryq.dev/en/stable/getting-started/backends-and-brokers/redis.html#visibility-timeout
     BROKER_TRANSPORT_OPTIONS = {
-        'visibility_timeout': BUILD_TIME_LIMIT * 1.15,  # 15% more than the build time limit
+        'visibility_timeout': 18000, # 5 hours
     }
 
     CELERY_DEFAULT_QUEUE = "celery"

Original file line number	Diff line number	Diff line change
`@@ -640,7 +640,7 @@ def BUILD_MEMORY_LIMIT(self):`
`640`	`640`	`# https://github.com/readthedocs/readthedocs.org/issues/12317#issuecomment-3070950434`
`641`	`641`	`# https://docs.celeryq.dev/en/stable/getting-started/backends-and-brokers/redis.html#visibility-timeout`
`642`	`642`	`BROKER_TRANSPORT_OPTIONS = {`
`643`		`- 'visibility_timeout': BUILD_TIME_LIMIT * 1.15, # 15% more than the build time limit`
	`643`	`+ 'visibility_timeout': 18000, # 5 hours`
`644`	`644`	`}`
`645`	`645`
`646`	`646`	`CELERY_DEFAULT_QUEUE = "celery"`