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

Enable TCP keepalives for render_list connections to renderd #271

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

Conversation

stephan2012
Copy link
Contributor

@stephan2012 stephan2012 commented Jan 10, 2022

This PR allows enabling and customizing TCP keepalives to better cope with load balancers silently closing connections.

Feedback is welcome.

Fixes #270

Signed-off-by: Stephan Austermühle <[email protected]>
Signed-off-by: Stephan Austermühle <[email protected]>
@pantierra pantierra marked this pull request as ready for review February 7, 2022 20:19
@pantierra
Copy link
Collaborator

@stephan2012 do you consider this ready for review?

@stephan2012
Copy link
Contributor Author

@stephan2012 do you consider this ready for review?

Yes, this PR is ready for review.

@pantierra
Copy link
Collaborator

Thanks! This looks good. Please catch latest master and I think this is ready to be merged.

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.

Enable keepalives for TCP connections to renderd
2 participants