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

Creates a new configuration file in rsyslog.d directory with any content. #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jefomin
Copy link

@jefomin jefomin commented Dec 10, 2019

If necessary, creates a new configuration file with any content that your
written in new config file - /etc/rsyslog.d/app.conf directory in the /etc/rsyslog.d directory

…n new config file - /etc/rsyslog.d/app.conf directory in the /etc/rsyslog.d directory

pillars:
rsyslog:
rsyslogd:
Copy link
Member

@fpytloun fpytloun Dec 10, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would consider renaming this key to custom or something to be able to determine it's custom config, something extra and not native feature of this formula..

In general we don't like these things much - states should have supported set of features and should not accept entering direct service configuration from model (pillar) as it cannot quarantee it will behave as user wants. But there are exceptions in some formulas, especially those that can't cover all possible use-cases so I am fine with this change for rsyslog.

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.

2 participants