search

sous-chefs / rsyslog

Development repository for the rsyslog cookbook
https://supermarket.chef.io/cookbooks/rsyslog
Apache License 2.0
65 stars 199 forks source link

[COOK-3608] #34

Closed atomic-penguin closed 11 years ago

atomic-penguin commented 11 years ago

The split mail.* facilities in the 50-default.conf template is a problem for me on mail servers. This seems to be an opinionated addition to Debian-ish rsyslog defaults as shipped, however I retained that functionality on non-RHEL systems for cookbook backwards compatibility.

Also, the 50-default.conf is very much geared towards Ubuntu/Debian. Which is not a huge problem, but the cookbook generates log files which will never get rotated on RHEL family, and probably other, systems.

I am adding a hash knob to allow log facilities and log destination to be rendered in the 50-default.conf template for these two requirements.

sethvargo commented 11 years ago

@atomic-penguin there are actually specs for this cookbook (I know, right?!) and they need updating :smile:. Please re-mark as Fix Provided when you have a chance to look at that.

atomic-penguin commented 11 years ago

Dammit @sethvargo, thats not even a useful test. Matches content 'Default rules for rsyslog.'

sethvargo commented 11 years ago

:+1:

sethvargo commented 11 years ago

Merged in f46584e