Skip to content

Conversation

hashi-derek
Copy link
Contributor

Manual backport of #19339

)

Add grpc keepalive configuration. (#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
@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
@hc-github-team-consul-core
Copy link
Collaborator

🤔 This PR has changes in the website/ directory but does not have a type/docs-cherrypick label. If the changes are for the next version, this can be ignored. If they are updates to current docs, attach the label to auto cherrypick to the stable-website branch after merging.

@hashi-derek hashi-derek merged commit d0fea44 into release/1.15.x Oct 24, 2023
@hashi-derek hashi-derek deleted the derekm/grpc-server-keepalive-1.15.x branch October 24, 2023 13:52
@hashi-derek hashi-derek changed the title Backport of Add grpc keepalive configuration into release/1.15.x (#19… [NET-6295] Backport of Add grpc keepalive configuration into release/1.15.x (#19… Oct 24, 2023
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.

3 participants