fluentd icon indicating copy to clipboard operation
fluentd copied to clipboard

feat: add umask configuration support

Open wanglu-cell opened this issue 7 months ago • 2 comments

Which issue(s) this PR fixes: Fixes #

What this PR does / why we need it: 增加 umask 配置支持,使日志文件权限控制更灵活。当用户需要指定非默认文件权限时,可通过配置 umask 参数实现。 Docs Changes: 无 Release Note: 新增功能: 支持通过 umask 参数控制输出文件权限

wanglu-cell avatar May 20 '25 15:05 wanglu-cell

Thanks for your contribution!

Could you please clarify what this PR does?

Fluentd already has umask command line option.

  • https://docs.fluentd.org/deployment/command-line-option

The limitation of this option is:

  • There is no system config option yet.
  • It works only when using --daemon or --no-supervisor.
    • For example, it does not work when using Docker.

daipom avatar May 21 '25 05:05 daipom

如果您添加了新的配置项,请同步更新配置文档。

新增功能:支持通过 umask 参数控制输出文件权限。

对于这类行为,建议在官方文档中予以明确说明。

此外,请遵循 DCO 要求,在提交记录(commit)中添加 Signed-off-by 行。

cosmo0920 avatar May 27 '25 07:05 cosmo0920

This PR has been automatically marked as stale because it has been open 30 days with no activity. Remove stale label or comment or this PR will be closed in 7 days

github-actions[bot] avatar Jun 26 '25 10:06 github-actions[bot]

This PR was automatically closed because of stale in 7 days

github-actions[bot] avatar Jul 03 '25 10:07 github-actions[bot]