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.

Sign up for GitHub

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

Jump to bottom

Increase max number of connections to the proxy #18

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Increase max number of connections to the proxy #18

wants to merge 3 commits into from

Conversation

kivutar
Copy link
Contributor

@kivutar kivutar commented Dec 10, 2018

No description provided.

@sihoang
Copy link
Member

sihoang commented Jan 2, 2019

why do we need this pam_limits.so? I don't understand this use case. Please elaborate

@kivutar
Copy link
Contributor Author

kivutar commented Jan 3, 2019

I'm not really sure about this PR. I've just read some optimization for nginx there and there on the web.
But so far, we don't really need it.

It's only needed when you want to support bursts with thouthands of requests per seconds.

@sihoang
Copy link
Member

sihoang commented Jan 3, 2019

I see. I think it's fine to pump the worker_connections to 10k in the nginx.conf but I would not touch the pam module though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kivutar @sihoang