PR#1261: waiverdb: Set fixed number of processes ans threads - fedora-infra/ansible

fedora-infra / ansible

#1261 waiverdb: Set fixed number of processes ans threads

Merged a year ago by zlopez. Opened a year ago by lholecek.

fedora-infra/ lholecek/ansible fix-waiverdb-memory-usage into main

waiverdb: Set fixed number of processes ans threads

Lukas Holecek • a year ago

c6df495

roles/openshift-apps/waiverdb/templates/deploymentconfig.yml

file modified

		`@@ -60,6 +60,11 @@`
		`mountPath: /etc/fedora-messaging`
		`readOnly: true`
		`env:`
		`+ - name: GUNICORN_CMD_ARGS`
		`+ value: >-`
		`+ --workers=3`
		`+ --threads=6`
		`+ --timeout=90`
		`- name: DATABASE_PASSWORD`
		`valueFrom:`
		`secretKeyRef:`

lholecek commented a year ago

New waiverdb container image starts 8 gunicorn web worker processes by
default using WEB_CONCURRENCY environment variable. This causes memory
to spike (over 500MiB) and workers to get terminated.

Instead of increasing memory limits, a better solution is to decrease
the number of processes and increase number of threads since the app
mostly waits on DB requests to finish and waiverdb workers themselves
are thread-safe.