opendev/puppet-exim
Code Issues Proposed changes

Align exim module parameter name with the tree

Browse Source 
The rest of the tree, literally everwhere, calls this value
sysadmins - but the exim module calls it sysadmin. This doesn't
really matter of course, but I noticed it while looking in to a
review on another patch and couldn't not fix it. Also, there are
two comments that are clearly remnants from having used exim.pp
as a starting point.

Change-Id: I184701c8c4d4b1e9d93d746b2ad0c5fc53443866
This commit is contained in:
Monty Taylor 2014-06-14 08:07:02 -07:00
parent 776945346f
commit c45f179da0
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
manifests/init.pp
View File

@ -4,7 +4,7 @@ class exim(
$queue_run_max = '5', $queue_run_max = '5',
$queue_smtp_domains = '', $queue_smtp_domains = '',
$smarthost = false, $smarthost = false,
$sysadmin = [] $sysadmins = []
) { ) {
include exim::params include exim::params

4
templates/aliases.erb
View File

@ -14,6 +14,6 @@ security: root
gerrit2: root gerrit2: root
jenkins: root jenkins: root
<% if sysadmin.length > 0 -%> <% if sysadmins.length > 0 -%>
root: <%= sysadmin.join(",") %> root: <%= sysadmins.join(",") %>
<% end -%> <% end -%>