Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Submit feedback
Sign in
Toggle navigation
H
heroku-buildpack-nodejs
Project
Project
Details
Activity
Releases
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
JIRA
JIRA
Merge Requests
0
Merge Requests
0
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Commits
Issue Boards
Open sidebar
Python-Dev
heroku-buildpack-nodejs
Commits
34cb60e7
Commit
34cb60e7
authored
Jul 07, 2016
by
Hunter Loftis
Committed by
GitHub
Jul 07, 2016
Browse files
Options
Browse Files
Download
Plain Diff
Merge pull request #308 from heroku/remove-32-concurrency-limit
remove hard max of 32 processes
parents
75ac1923
2cb88f84
Changes
2
Hide whitespace changes
Inline
Side-by-side
Showing
2 changed files
with
0 additions
and
9 deletions
+0
-9
nodejs.sh
profile/nodejs.sh
+0
-2
run
test/run
+0
-7
No files found.
profile/nodejs.sh
View file @
34cb60e7
...
...
@@ -4,8 +4,6 @@ calculate_concurrency() {
WEB_CONCURRENCY
=
${
WEB_CONCURRENCY
-
$((
MEMORY_AVAILABLE/WEB_MEMORY
))}
if
((
WEB_CONCURRENCY < 1
))
;
then
WEB_CONCURRENCY
=
1
elif
((
WEB_CONCURRENCY
>
32
))
;
then
WEB_CONCURRENCY
=
32
fi
WEB_CONCURRENCY
=
$WEB_CONCURRENCY
}
...
...
test/run
View file @
34cb60e7
...
...
@@ -192,13 +192,6 @@ testConcurrencyCustomLimit() {
assertCapturedSuccess
}
testConcurrencySaneMaximum
()
{
LOG_CONCURRENCY
=
true
MEMORY_AVAILABLE
=
6144
WEB_MEMORY
=
32 capture
$(
pwd
)
/profile/nodejs.sh
assertCaptured
"Detected 6144 MB available memory, 32 MB limit per process (WEB_MEMORY)"
assertCaptured
"Recommending WEB_CONCURRENCY=32"
assertCapturedSuccess
}
testInvalidNode
()
{
compile
"invalid-node"
assertCaptured
"Downloading and installing node 0.11.333"
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment