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

Unified 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.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: modules/nginx/manifests/params.pp
===================================================================
--- a/modules/nginx/manifests/params.pp
+++ b/modules/nginx/manifests/params.pp
@@ -1,9 +1,11 @@
class nginx::params {
+ $worker_processes = 2
+ $worker_connections = 1024
$multi_accept = off
$worker_rlimit_nofile = 30000
$sendfile = on
$keepalive_timeout = 15
$tcp_nodelay = on
$gzip = on
$user = $::operatingsystem ? {
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld