Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Side by Side Diff: manifests/filterserver.pp

Issue 5302563972841472: Merge filter and notification servers, have both tasks run on the same servers (Closed)
Patch Set: Stopped misusing is_default parameter Created Jan. 30, 2014, 2:10 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | manifests/monitoringserver.pp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 node 'server5', 'server6', 'server7', 'server11', 'server12', 'server15', 'serve r19', 'filter1', 'filter2', 'filter3', 'filter4', 'filter5', 'filter6' { 1 node 'server5', 'server6', 'server7', 'server11', 'server12', 'server15', 'serve r19', 'filter1', 'filter2', 'filter3', 'filter4', 'filter5', 'filter6', 'notific ation1', 'notification2' {
2 include base, filterserver 2 include base
3 3
4 class {'filterserver':
5 is_default => true
6 }
7
8 class {'notificationserver':
9 is_default => false
Felix Dahlke 2014/01/30 14:31:29 I'd prefer not to explicitly set this to false her
Wladimir Palant 2014/01/30 14:50:23 I'd rather keep invoking both modules the same way
10 }
11
4 class {'nagios::client': 12 class {'nagios::client':
5 server_address => 'monitoring.adblockplus.org' 13 server_address => 'monitoring.adblockplus.org'
6 } 14 }
7 15
8 class {'statsclient': } 16 class {'statsclient': }
9 } 17 }
OLDNEW
« no previous file with comments | « no previous file | manifests/monitoringserver.pp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld