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

[NET-6295] Backport of Add grpc keepalive configuration. (#19339) #19355

Merged
merged 1 commit into from
Oct 24, 2023

Conversation

hashi-derek
Copy link
Member

Manual backport of #19339

Prior to the introduction of this configuration, grpc keepalive messages were
sent after 2 hours of inactivity on the stream. This posed issues in various
scenarios where the server-side xds connection balancing was unaware that envoy
instances were uncleanly killed / force-closed, since the connections would
only be cleaned up after ~5 minutes of TCP timeouts occurred. Setting this
config to a 30 second interval with a 20 second timeout ensures that at most,
it should take up to 50 seconds for a dead xds connection to be closed.
@hashi-derek hashi-derek added pr/no-changelog PR does not need a corresponding .changelog entry pr/no-backport labels Oct 24, 2023
@hashi-derek hashi-derek requested review from a team as code owners October 24, 2023 19:21
@github-actions github-actions bot added type/docs Documentation needs to be created/updated/clarified theme/config Relating to Consul Agent configuration, including reloading labels Oct 24, 2023
@hashi-derek hashi-derek changed the title Backport of Add grpc keepalive configuration. (#19339) [NET-6295] Backport of Add grpc keepalive configuration. (#19339) Oct 24, 2023
@hashi-derek hashi-derek merged commit fe5e19c into release/1.17.0 Oct 24, 2023
95 checks passed
@hashi-derek hashi-derek deleted the derekm/grpc-server-keepalive-1.17.0 branch October 24, 2023 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/no-backport pr/no-changelog PR does not need a corresponding .changelog entry theme/config Relating to Consul Agent configuration, including reloading type/docs Documentation needs to be created/updated/clarified
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants