Skip to content

Comments

out_forward: add send_keepalive_packet parameter#608

Open
Watson1978 wants to merge 1 commit intofluent:1.0from
Watson1978:send_keepalive_packet
Open

out_forward: add send_keepalive_packet parameter#608
Watson1978 wants to merge 1 commit intofluent:1.0from
Watson1978:send_keepalive_packet

Conversation

@Watson1978
Copy link
Contributor

@Watson1978 Watson1978 commented Feb 24, 2026

fluent/fluentd#5262

The above PR adds new feature into Fluentd 1.20.0.
So, when release Fluentd 1.20.0, we need to merge this document changes.

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
Watson1978 pushed a commit to fluent/fluentd that referenced this pull request Feb 24, 2026
**Which issue(s) this PR fixes**: 
Fixes #

**What this PR does / why we need it**: 

This PR adds the `send_keepalive_packet` configuration parameter to the
`out_forward` plugin to enable TCP keepalive (SO_KEEPALIVE) on forward
output connections.

**Background:**
The existing `keepalive` parameter only enables connection
pooling/reuse, not TCP-level keepalive probes. This can lead to
accumulation of dead connections in CLOSE-WAIT state.

**Changes:**
- Added `send_keepalive_packet` config parameter (default: false)
- Pass the parameter to both `socket_create_tcp` and `socket_create_tls`
calls
- Added tests to verify the parameter works correctly

**Related:**
This follows the same pattern as #2352 which added the same feature to
`in_forward`.

**Docs Changes**:
fluent/fluentd-docs-gitbook#608

**Release Note**: 

* out_forward: Add `send_keepalive_packet` option

Signed-off-by: Ian Driver <idriver@williamhill.us>
Co-authored-by: Ian Driver <idriver@williamhill.us>
@Watson1978 Watson1978 added the pending To be done in the future label Feb 24, 2026
@Watson1978 Watson1978 requested a review from kenhys February 24, 2026 05:27
Copy link
Contributor

@kenhys kenhys left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

pending now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pending To be done in the future

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants