Skip to content

Conversation

daipom
Copy link
Contributor

@daipom daipom commented Jul 28, 2025

Which issue(s) this PR fixes:

What this PR does / why we need it:
Add use_logger option to out_stdout plugin.

  • Type: bool
  • Default: true

If you set false to this option, out_stdout outputs events to $stdout, instead of $logger.
This feature enables events to be output to STDOUT in containerized environments, such as K8s.

This does not change the default behavior.

Note: Although it's preferable for the default value of a bool option to be false (because it can be used as a flag), I couldn't think of a suitable name for it.

Docs Changes:
fluent/fluentd-docs-gitbook#593

Release Note:
out_stdout: support output to STDOUT independently of Fluentd logger by setting use_logger to false.

@daipom daipom added this to the v1.19.0 milestone Jul 29, 2025
@daipom daipom force-pushed the out_stdout-add-feature-sync_logger branch from 3e14f1d to 203627e Compare July 29, 2025 03:53
Signed-off-by: Daijiro Fukuda <fukuda@clear-code.com>
Co-authored-by: Takuro Ashie <ashie@clear-code.com>
Co-authored-by: Kentaro Hayashi <hayashi@clear-code.com>
Co-authored-by: Shizuo Fujita <fujita@clear-code.com>
@daipom daipom force-pushed the out_stdout-add-feature-sync_logger branch from 203627e to a82db7a Compare July 29, 2025 03:54
daipom added a commit to daipom/fluentd-docs-gitbook that referenced this pull request Jul 29, 2025
fluent/fluentd#5036

Signed-off-by: Daijiro Fukuda <fukuda@clear-code.com>
@daipom daipom marked this pull request as ready for review July 29, 2025 04:06
Watson1978
Watson1978 previously approved these changes Jul 29, 2025
Copy link
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.

👍🏻

Signed-off-by: Daijiro Fukuda <fukuda@clear-code.com>

Co-authored-by: Kentaro Hayashi <kenhys@gmail.com>
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.

@daipom
Copy link
Contributor Author

daipom commented Jul 30, 2025

Thanks!
Some tests for Windows are abnormally slow, but it is not related to this PR (It has been occurring for a month).

@daipom daipom merged commit 1f64300 into fluent:master Jul 30, 2025
28 of 29 checks passed
@daipom daipom deleted the out_stdout-add-feature-sync_logger branch July 30, 2025 04:37
daipom added a commit to fluent/fluentd-docs-gitbook that referenced this pull request Jul 30, 2025
fluent/fluentd#5036

Signed-off-by: Daijiro Fukuda <fukuda@clear-code.com>
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.

Output to Log file changes the destination of stdout output plugin too
3 participants