Threaded gunicorn workers so a retry storm cannot starve the UI
This commit is contained in:
@@ -139,10 +139,16 @@ HEALTHCHECK --interval=30s --timeout=5s --start-period=10s --retries=3 \
|
|||||||
-H "Content-Type: application/json" -d '{}' \
|
-H "Content-Type: application/json" -d '{}' \
|
||||||
| grep -q "VALIDATION_ERROR" || exit 1
|
| grep -q "VALIDATION_ERROR" || exit 1
|
||||||
|
|
||||||
|
# Threaded workers, matching docker-compose.yml - see the reasoning there. A
|
||||||
|
# compose file overrides this CMD, but an image run without one (a plain
|
||||||
|
# `docker run`, a site's own orchestrator) would otherwise still get the
|
||||||
|
# blocking sync default and the same starvation.
|
||||||
CMD ["gunicorn", \
|
CMD ["gunicorn", \
|
||||||
"--bind", "0.0.0.0:5001", \
|
"--bind", "0.0.0.0:5001", \
|
||||||
|
"--worker-class", "gthread", \
|
||||||
"--workers", "4", \
|
"--workers", "4", \
|
||||||
"--timeout", "60", \
|
"--threads", "8", \
|
||||||
|
"--timeout", "30", \
|
||||||
"--access-logfile", "-", \
|
"--access-logfile", "-", \
|
||||||
"--error-logfile", "-", \
|
"--error-logfile", "-", \
|
||||||
"wsgi:app"]
|
"wsgi:app"]
|
||||||
|
|||||||
@@ -79,10 +79,26 @@ services:
|
|||||||
- gunicorn
|
- gunicorn
|
||||||
- --bind
|
- --bind
|
||||||
- 0.0.0.0:5001
|
- 0.0.0.0:5001
|
||||||
|
# Threaded workers, not the blocking sync default. An agent that fails
|
||||||
|
# the TLS handshake retries in a loop, and with 4 sync workers a handful
|
||||||
|
# of those pinned every worker: real UI requests queued 20+ seconds while
|
||||||
|
# the CPU sat idle. A stuck connection now holds one THREAD, and 32
|
||||||
|
# (4 x 8) have to be exhausted before a person waits.
|
||||||
|
#
|
||||||
|
# Found in production at the other site (their commit db75ca7). Their
|
||||||
|
# server, our agent - so the same client behaviour will reach any site
|
||||||
|
# running the endpoint agent, and the fix belongs in the product rather
|
||||||
|
# than on one branch.
|
||||||
|
- --worker-class
|
||||||
|
- gthread
|
||||||
- --workers
|
- --workers
|
||||||
- "4"
|
- "4"
|
||||||
|
- --threads
|
||||||
|
- "8"
|
||||||
|
# 30, not 60: a hung connection frees its thread in half the time, which
|
||||||
|
# is what decides whether a retry storm is absorbed or queues.
|
||||||
- --timeout
|
- --timeout
|
||||||
- "60"
|
- "30"
|
||||||
- --access-logfile
|
- --access-logfile
|
||||||
- "-"
|
- "-"
|
||||||
- --error-logfile
|
- --error-logfile
|
||||||
|
|||||||
Reference in New Issue
Block a user