Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for CACHES. #467

Merged
merged 1 commit into from
May 22, 2021
Merged

Add support for CACHES. #467

merged 1 commit into from
May 22, 2021

Conversation

maxking
Copy link
Owner

@maxking maxking commented May 21, 2021

No description provided.

Also, bump Django to 3.1.

[[[[
@maxking maxking changed the title caches Add support for CACHES. Also, bump Django to 3.1. May 21, 2021
@maxking maxking changed the title Add support for CACHES. Also, bump Django to 3.1. Add support for CACHES. May 21, 2021
@danil-smirnov
Copy link
Contributor

Should we also move to multi-process mode for uwsgi @maxking ?

https://github.com/maxking/docker-mailman/blob/main/web/mailman-web/uwsgi.ini#L17

@maxking maxking merged commit 496ee0c into main May 22, 2021
@maxking maxking deleted the caches branch May 22, 2021 16:42
@maxking
Copy link
Owner Author

maxking commented May 22, 2021

@danil-smirnov yeah, I think we can safely bump the number of processes now.

@danil-smirnov
Copy link
Contributor

@maxking I'm not sure what processes/threads configuration is reasonable actually... I didn't hit any issue with the 1/2 setup. Do we really need to use multiple processes here?...

@maxking
Copy link
Owner Author

maxking commented May 23, 2021

It really depends on how much traffic do you get on your webserver. I think 2 workers with 2 threads each should suffice for most people. It might be good to have 2 workers to take advantage of 2 Cores and then spin off 2 threads each (threads are much cheaper resources wise, compared to workers) so we have some headroom for high traffic times.

@danil-smirnov
Copy link
Contributor

Then let's apply it @maxking :)
#470

@danil-smirnov
Copy link
Contributor

I wonder if Postorius image should have the same configuration @maxking ?
The current four processes look a bit overkill to me...

@maxking
Copy link
Owner Author

maxking commented May 30, 2021

We can probably do the same with Postorius, its not that much of an overhead.

@danil-smirnov
Copy link
Contributor

@maxking Then this is it: #472

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

Successfully merging this pull request may close these issues.

2 participants