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

Enabled custom 35-server-per-host.conf templates #174

Merged
merged 4 commits into from
Nov 23, 2020

Conversation

axl89
Copy link
Contributor

@axl89 axl89 commented Oct 16, 2020

Description

This enables a complete or partial override of the 35-server-per-host.conf file.

For instance, disable debug and/or messages, etc.

Issues Resolved

#173

Check List

  • All tests pass. See TESTING.md for details.
  • New functionality includes testing.
  • New functionality has been documented in the README if applicable.

This enables a complete or partial override of the settings. For
instance, disable debug and/or messages, etc.
@axl89
Copy link
Contributor Author

axl89 commented Oct 16, 2020

My plan is to create a wrapper cookbook and set a different template, with the following differences from the original:

diff --git a/templates/default/35-server-per-host.conf.erb b/templates/default/35-server-per-host.conf.erb
index 91cbcc5..3def571 100644
--- a/templates/default/35-server-per-host.conf.erb
+++ b/templates/default/35-server-per-host.conf.erb
@@ -43,14 +43,6 @@ news.crit               ?PerHostNewsCrit
 news.err                ?PerHostNewsErr
 news.notice             -?PerHostNewsNotice
 
-*.=debug;\
-  auth,authpriv.none;\
-  news.none;mail.none   -?PerHostDebug
-
-*.=info;*.=notice;*.=warn;\
-  auth,authpriv.none;\
-  cron,daemon.none;\
-  mail,news.none        -?PerHostMessages
 
 
 <% unless node['rsyslog']['allow_non_local'] -%>

@axl89
Copy link
Contributor Author

axl89 commented Oct 16, 2020

Tests are failing, but they are also failing on master.

@ramereth
Copy link
Contributor

@axl89 this cookbook was just transferred to us and we're still in the process of getting it properly imported into Sous Chefs. Please give us a week or so to get ready and we'll have you rebase.

@xorima xorima added the Release: Minor Release to Chef Supermarket as a minor release when merged label Nov 23, 2020
@ramereth ramereth merged commit 0352f6c into sous-chefs:master Nov 23, 2020
@kitchen-porter
Copy link
Contributor

Released as: 7.3.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release: Minor Release to Chef Supermarket as a minor release when merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants