Skip to content

docs(in_exec): add command_timeout parameter documentation#619

Open
zoklk wants to merge 2 commits intofluent:1.0from
zoklk:1.0
Open

docs(in_exec): add command_timeout parameter documentation#619
zoklk wants to merge 2 commits intofluent:1.0from
zoklk:1.0

Conversation

@zoklk
Copy link
Copy Markdown

@zoklk zoklk commented Apr 10, 2026

Add documentation for command_timeout option in in_exec plugin.

Note: The version field is set to 1.19.0 based on fluentd/lib/fluent/version.rb.
Please update to the appropriate release version before merging.

Related: fluent/fluentd#5319

Signed-off-by: zoklk <bjincheol34@gmail.com>
Comment thread input/exec.md Outdated
Co-authored-by: Shizuo Fujita <watson1978@gmail.com>
Signed-off-by: zoklk <bjincheol34@gmail.com>
kenhys pushed a commit to fluent/fluentd that referenced this pull request Apr 16, 2026
#5320)

**Which issue(s) this PR fixes**:
Fixes #5319

**What this PR does / why we need it**:
Adds `command_timeout` option to `in_exec` plugin, mirroring the
existing
`command_timeout` option in `out_exec`.

Currently, `in_exec` calls `child_process_execute` without
`wait_timeout`,
defaulting to `nil` (infinite wait). When the external command hangs,
Fluentd repeatedly shows `previous child process is still running.
skipped.`
with no indication of the actual cause, making troubleshooting
difficult.

`out_exec` already exposes `command_timeout` which maps to
`wait_timeout`
in `child_process_execute`. This PR resolves the inconsistency between
the two plugins.

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

**Release Note**:
Add `command_timeout` option to `in_exec` plugin to kill long-running
child processes and align behavior with `out_exec`.

---------

Signed-off-by: zoklk <bjincheol34@gmail.com>
Copy link
Copy Markdown
Contributor

@Watson1978 Watson1978 left a comment

Choose a reason for hiding this comment

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

Thanks

@Watson1978 Watson1978 added the pending To be done in the future label Apr 16, 2026
@Watson1978
Copy link
Copy Markdown
Contributor

Should be merged after Fluentd 1.20 or later

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