manifests/filterserver.pp - Issue 9343001: Add Postfix configuration to the repository, make Postfix listen onlocal interfaces only

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments.

Keyboard Shortcuts

	File
u :	up to issue
m :	publish + mail comments
M :	edit review message
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line
<Enter> :	respond to / edit current comment
d :	mark current comment as done

	Issue
u :	up to list of issues
m :	publish + mail comments
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue
# :	close issue

	Comment/message editing
<Ctrl> + s or <Ctrl> + Enter :	save comment
<Esc> :	cancel edit

Side by Side Diff: manifests/filterserver.pp

Issue 9343001: Add Postfix configuration to the repository, make Postfix listen onlocal interfaces only (Closed)

Patch Set: Created Feb. 6, 2013, 8:19 a.m.

Left:
Right:

Use n/p to move between diff chunks; N/P to move between comments.

Jump to:

View unified diff | Download patch

OLD	NEW
1 node 'server1', 'server3', 'server5', 'server6', 'server7', 'server8', 'server9' {	1 node 'server1', 'server3', 'server5', 'server6', 'server7', 'server8', 'server9' {

2 include base, filterserver	2 include base, postfix, filterserver
	Felix Dahlke 2013/02/06 09:17:58 Shouldn't we include postfix in base? We need it o Shouldn't we include postfix in base? We need it on every system to send cron mails, and every system has cron jobs of whose failures we want to be informed. Wladimir Palant 2013/02/06 09:58:39 Same is true for SSH - but whatever :) Show quoted text On 2013/02/06 09:17:58, Felix H. Dahlke wrote: > Shouldn't we include postfix in base? We need it on every system to send cron > mails, and every system has cron jobs of whose failures we want to be informed. Same is true for SSH - but whatever :) Felix Dahlke 2013/02/06 10:02:06 Yes, but the problem with SSH is that it has an op Show quoted text On 2013/02/06 09:58:39, Wladimir Palant wrote: > On 2013/02/06 09:17:58, Felix H. Dahlke wrote: > > Shouldn't we include postfix in base? We need it on every system to send cron > > mails, and every system has cron jobs of whose failures we want to be > informed. > > Same is true for SSH - but whatever :) Yes, but the problem with SSH is that it has an optional parameter that some nodes set, others not, and we can't include it twice.
3	3

4 class {'nagios::client':	4 class {'nagios::client':

5 server_address => 'monitoring.adblockplus.org'	5 server_address => 'monitoring.adblockplus.org'

6 }	6 }

7 }	7 }

OLD	NEW

« no previous file with comments | « no previous file | manifests/monitoringserver.pp » ('j') | no next file with comments »