Skip to content
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

feat: add support for file and directory mode/owner/group for output files #400

Merged
merged 1 commit into from
Jul 22, 2024

Conversation

richm
Copy link
Collaborator

@richm richm commented Jul 16, 2024

Feature: Add support for specifying the mode, owner, and group for files
and directories created by logging_output type files. The new variables
are mode, owner, group, dir_mode, dir_owner, dir_group and
these correspond to the Ansible file et. al. modules parameters.

Reason: Users must be able to set ownership/permissions for files and
directories created by rsyslog.

Result: Users are able to set ownership/permissions for files and
directories created by rsyslog.

Signed-off-by: Rich Megginson [email protected]

@richm richm changed the title file dir mode owner group feat: add support for file and directory mode/owner/group for output files Jul 16, 2024
@richm
Copy link
Collaborator Author

richm commented Jul 16, 2024

[citest]

…files

Feature: Add support for specifying the mode, owner, and group for files
and directories created by `logging_output` type `files`.  The new variables
are `mode`, `owner`, `group`, `dir_mode`, `dir_owner`, `dir_group` and
these correspond to the Ansible `file` et. al. modules parameters.

Reason: Users must be able to set ownership/permissions for files and
directories created by rsyslog.

Result: Users are able to set ownership/permissions for files and
directories created by rsyslog.

Signed-off-by: Rich Megginson <[email protected]>
@richm richm force-pushed the file-dir-mode-owner-group branch from 223acb6 to c94f762 Compare July 22, 2024 13:59
@richm richm merged commit 9963be4 into linux-system-roles:main Jul 22, 2024
8 checks passed
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.

1 participant