-
Notifications
You must be signed in to change notification settings - Fork 1.4k
System config: add umask option #4816
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Labels
Comments
egemenkus
added a commit
to egemenkus/fluentd
that referenced
this issue
Feb 10, 2025
Previously, Fluentd users had to use the --umask command-line argument to set the umask, which was inconvenient for service deployments and containerized environments. This commit introduces a umask option in the Fluentd system configuration, allowing users to define umask directly within fluent.conf. Additionally, this commit fixes test cases related to umask handling: - The previous tests incorrectly asserted the expected umask values. - Adjusted tests to ensure that new file permissions are correctly applied according to the configured umask. - Added a test to verify that an invalid umask value raises the expected Fluent::ConfigError. Fixes fluent#4816 Release Note: - Added umask option to Fluentd system configuration. Signed-off-by: kushynoda <egemen.utku3@gmail.com>
It would be necessary for us to handle the following PR first: |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is your feature request related to a problem? Please describe.
Some users need to change
umask
.Example:
Currently, we have to use the command line option
--umask
.It is inconvenient for the service or container image.
Describe the solution you'd like
Add umask option to the System config.
Describe alternatives you've considered
Without this option in System config, we can customize the command line options like #4810 (comment), but it is inconvenient.
Additional context
No response
The text was updated successfully, but these errors were encountered: