FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

docs(in_exec): add command_timeout parameter documentation by zoklk · Pull Request #619 · fluent/fluentd-docs-gitbook · GitHub

Repository navigation

docs(in_exec): add command_timeout parameter documentation - #619

Merged
Watson1978 merged 2 commits into
fluent:1.0from
zoklk:1.0
Oct 23, 2026
Merged

Watson1978 merged 2 commits into
fluent:1.0from
zoklk:1.0

Conversation

zoklk commented Apr 10, 2026

Copy link
Copy Markdown
Contributor

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>

Watson1978 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Thanks

Watson1978 added the pending To be done in the future label Apr 16, 2026

Copy link
Copy Markdown
Contributor

Should be merged after Fluentd 1.20 or later

kenhys added this to the fluentd-v1.20 milestone Jul 16, 2026
Watson1978 removed the pending To be done in the future label Oct 23, 2026
Watson1978 merged commit 60c9ade into fluent:1.0 Oct 23, 2026
2 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL