Skip to content

system_config: add umask option to <system> directive#5390

Open
somaz94 wants to merge 1 commit into
fluent:masterfrom
somaz94:feat/system-config-umask
Open

system_config: add umask option to <system> directive#5390
somaz94 wants to merge 1 commit into
fluent:masterfrom
somaz94:feat/system-config-umask

Conversation

@somaz94

@somaz94 somaz94 commented Jun 22, 2026

Copy link
Copy Markdown

Which issue(s) this PR fixes: Fixes #4816

What this PR does / why we need it:
Adds a umask option to the <system> directive so the process umask can be
set from the config file. Previously this required the --umask command line
option, which is inconvenient for services and container images.

It reuses the existing chumask handling: <system> umask sets the same value
that --umask does, and the command line option takes precedence when both are
given. Both supervised and --no-supervisor modes honor it.

<system>
  umask 0022
</system>

Tests: added a config round-trip case and a supervisor test asserting the value
reaches ServerEngine, including command-line precedence.

Docs Changes:
Follow-up in fluent/fluentd-docs-gitbook (the <system> directive docs).

Release Note:
Add umask option to the <system> directive.

Signed-off-by: somaz <genius5711@gmail.com>
@somaz94 somaz94 marked this pull request as ready for review June 23, 2026 09:40
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.

System config: add umask option

1 participant