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

Side by Side Diff: modules/nginx/manifests/params.pp

Issue 9316104: Reduced the number of nginx worker processes to match the number ofCPU cores and removed unused va… (Closed)
Patch Set: Reverted the params.pp change - I didn`t realize that these values were actually used as defaults. Created Feb. 6, 2013, 12:08 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 | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 class nginx::params { 1 class nginx::params {
2 $worker_processes = 2
3 $worker_connections = 1024
2 $multi_accept = off 4 $multi_accept = off
3 $worker_rlimit_nofile = 30000 5 $worker_rlimit_nofile = 30000
4 $sendfile = on 6 $sendfile = on
5 $keepalive_timeout = 15 7 $keepalive_timeout = 15
6 $tcp_nodelay = on 8 $tcp_nodelay = on
7 $gzip = on 9 $gzip = on
8 10
9 $user = $::operatingsystem ? { 11 $user = $::operatingsystem ? {
10 /(?i-mx:debian|ubuntu)/ => 'www-data', 12 /(?i-mx:debian|ubuntu)/ => 'www-data',
11 /(?i-mx:fedora|rhel|redhat|centos|scientific|suse|opensuse)/ => 'nginx', 13 /(?i-mx:fedora|rhel|redhat|centos|scientific|suse|opensuse)/ => 'nginx',
12 } 14 }
13 15
14 $group = $::operatingsystem ? { 16 $group = $::operatingsystem ? {
15 /(?i-mx:debian|ubuntu)/ => 'www-data', 17 /(?i-mx:debian|ubuntu)/ => 'www-data',
16 /(?i-mx:fedora|rhel|redhat|centos|scientific|suse|opensuse)/ => 'www', 18 /(?i-mx:fedora|rhel|redhat|centos|scientific|suse|opensuse)/ => 'www',
17 } 19 }
18 } 20 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld