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

Side by Side Diff: modules/notificationserver/manifests/init.pp

Issue 29448574: Noissue - Use of spawn_fcgi module instead of spawn-fcgi (Closed)
Patch Set: Created May 25, 2017, 4:55 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 | « modules/nagios/manifests/server.pp ('k') | modules/trac/manifests/init.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 class notificationserver($is_default = false) { 1 class notificationserver($is_default = false) {
2 if !defined(Class['nginx']) { 2 if !defined(Class['nginx']) {
3 class {'nginx': 3 class {'nginx':
4 worker_connections => 4000, 4 worker_connections => 4000,
5 ssl_session_cache => off, 5 ssl_session_cache => off,
6 } 6 }
7 } 7 }
8 8
9 class {'sitescripts': 9 class {'sitescripts':
10 sitescriptsini_source => 'puppet:///modules/notificationserver/sitescripts.i ni' 10 sitescriptsini_source => 'puppet:///modules/notificationserver/sitescripts.i ni'
(...skipping 10 matching lines...) Expand all
21 } 21 }
22 22
23 cron {'update_notifications': 23 cron {'update_notifications':
24 command => 'hg pull -q -u -R /opt/notifications', 24 command => 'hg pull -q -u -R /opt/notifications',
25 environment => hiera('cron::environment', []), 25 environment => hiera('cron::environment', []),
26 minute => '2-59/20', 26 minute => '2-59/20',
27 user => 'nginx', 27 user => 'nginx',
28 require => Exec['fetch_notifications'], 28 require => Exec['fetch_notifications'],
29 } 29 }
30 30
31 include spawn-fcgi 31 include spawn_fcgi
32 32
33 spawn-fcgi::pool {'multiplexer': 33 spawn_fcgi::pool {'multiplexer':
34 ensure => present, 34 ensure => present,
35 fcgi_app => '/opt/sitescripts/multiplexer.fcgi', 35 fcgi_app => '/opt/sitescripts/multiplexer.fcgi',
36 socket => '/tmp/multiplexer-fastcgi.sock', 36 socket => '/tmp/multiplexer-fastcgi.sock',
37 mode => '0666', 37 mode => '0666',
38 order => 500, 38 order => 500,
39 user => 'nginx', 39 user => 'nginx',
40 children => 1, 40 children => 1,
41 require => [ 41 require => [
42 Class['sitescripts'], 42 Class['sitescripts'],
43 Exec['fetch_notifications'], 43 Exec['fetch_notifications'],
(...skipping 13 matching lines...) Expand all
57 nginx::hostconfig{'notification.adblockplus.org': 57 nginx::hostconfig{'notification.adblockplus.org':
58 source => 'puppet:///modules/notificationserver/site.conf', 58 source => 'puppet:///modules/notificationserver/site.conf',
59 global_config => template('notificationserver/global.conf.erb'), 59 global_config => template('notificationserver/global.conf.erb'),
60 is_default => $is_default, 60 is_default => $is_default,
61 certificate => 'easylist-downloads.adblockplus.org_sslcert.pem', 61 certificate => 'easylist-downloads.adblockplus.org_sslcert.pem',
62 private_key => 'easylist-downloads.adblockplus.org_sslcert.key', 62 private_key => 'easylist-downloads.adblockplus.org_sslcert.key',
63 log => 'access_log_notification', 63 log => 'access_log_notification',
64 log_format => 'notification', 64 log_format => 'notification',
65 } 65 }
66 } 66 }
OLDNEW
« no previous file with comments | « modules/nagios/manifests/server.pp ('k') | modules/trac/manifests/init.pp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld