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

Issue 9316104: Reduced the number of nginx worker processes to match the number ofCPU cores and removed unused va… (Closed)

Created:
Feb. 6, 2013, 8:38 a.m. by Wladimir Palant
Modified:
Nov. 8, 2013, 8:04 a.m.
Reviewers:
Andrey Novikov
Visibility:
Public.

Description

Reduced the number of nginx worker processes to match the number of CPU cores and removed unused va…

Patch Set 1 #

Patch Set 2 : Reverted the params.pp change - I didn`t realize that these values were actually used as defaults. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -0 lines) Patch
M modules/nginx/manifests/params.pp View 1 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 5
Wladimir Palant
Feb. 6, 2013, 8:38 a.m. (2013-02-06 08:38:13 UTC) #1
Andrey Novikov
LGTM
Feb. 6, 2013, 9:52 a.m. (2013-02-06 09:52:57 UTC) #2
Andrey Novikov
LGTM
Feb. 6, 2013, 9:53 a.m. (2013-02-06 09:53:21 UTC) #3
Wladimir Palant
Reverted the params.pp change - I didn`t realize that these values were actually used as ...
Feb. 6, 2013, 12:08 p.m. (2013-02-06 12:08:46 UTC) #4
Andrey Novikov
Feb. 6, 2013, 1:24 p.m. (2013-02-06 13:24:35 UTC) #5
Shame on me! I didn't notice the second file, otherwise I would tell you.

Powered by Google App Engine
This is Rietveld