Skip to content
This repository has been archived by the owner on Jun 16, 2021. It is now read-only.

switch to gunicorn #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

switch to gunicorn #5

wants to merge 1 commit into from

Conversation

ltalirz
Copy link
Member

@ltalirz ltalirz commented Sep 14, 2020

As pointed out in the README of the base image
https://github.com/tiangolo/uwsgi-nginx-flask-docker, one can gain up to
4x performance benefits by switching to
https://github.com/tiangolo/meinheld-gunicorn-flask-docker instead.

This switches to the gunicorn image, fixing the number of workers to two
(taking into account that the memory footprint of the container scales
essentially linearly with the number of workers requested).

As pointed out in the README of the base image
https://github.com/tiangolo/uwsgi-nginx-flask-docker, one can gain up to
4x performance benefits by switching to
https://github.com/tiangolo/meinheld-gunicorn-flask-docker instead.

This switches to the gunicorn image, fixing the number of workers to two
(taking into account that the memory footprint of the container scales
essentially linearly with the number of workers requested).
@ltalirz ltalirz force-pushed the issue_4_gunicorn_switch branch from fad44cf to 35685d5 Compare September 17, 2020 11:00
@ltalirz
Copy link
Member Author

ltalirz commented Sep 18, 2020

Unfortunately there is an issue blocking this
tiangolo/meinheld-gunicorn-flask-docker#38

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant