tonic: expose tcp_keepalive_interval and tcp_keepalive_retries options #2472
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add two new configuration methods to the Server builder:
tcp_keepalive_interval(): Sets TCP_KEEPINTVL socket optiontcp_keepalive_retries(): Sets TCP_KEEPCNT socket optionThese options were already implemented in TcpIncoming but not exposed through the Server builder API. This change allows users to fine-tune TCP keepalive behavior for scenarios like:
The new options default to None (system default) and only take effect when tcp_keepalive() is also configured.
Motivation
TcpIncomingalready supportswith_keepalive_interval()andwith_keepalive_retries(), but these are not exposed through theServerbuilder. Users need fine-grained control over TCP keepalive for cloud LB timeout matching and faster dead connection detection.Solution
Add
tcp_keepalive_interval()andtcp_keepalive_retries()methods toServerbuilder, passing them through toTcpIncominginbind_incoming(). Both default toNoneand only take effect whentcp_keepalive()is also set. Tests updated accordingly.