opendev/gerrit
Code Issues Proposed changes

Fix acceptance test heap size to 128m

Browse Source 
Buck defaults to running 1.25 * ncpus tests in parallel.  If each test
uses a heap size of 128M then an 8 core system will need at least
1.25G of RAM for JVM heap during test execution.  The actual memory
used will of course be much higher due to JVM overhead.

Set a default memory for acceptance tests to prevent a huge explosion
during `buck test --all`.

Change-Id: I373df822c029d2dc24c625129f540211cda910e7
This commit is contained in:
Shawn Pearce
2013-07-29 23:08:25 -07:00
committed by Dave Borowitz
parent 58920b3468
commit 79c58714de
2 changed files with 1 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/git/BUCK
View File

@@ -3,13 +3,11 @@ include_defs('//gerrit-acceptance-tests/tests.defs')
acceptance_tests( acceptance_tests(
srcs = ['SubmitOnPushIT.java'], srcs = ['SubmitOnPushIT.java'],
deps = [':util'], deps = [':util'],
vm_args = ['-Xmx512m'],
) )
acceptance_tests( acceptance_tests(
srcs = ['HttpPushForReviewIT.java', 'SshPushForReviewIT.java'], srcs = ['HttpPushForReviewIT.java', 'SshPushForReviewIT.java'],
deps = [':push_for_review'], deps = [':push_for_review'],
vm_args = ['-Xmx512m'],
) )
java_library( java_library(

2
gerrit-acceptance-tests/tests.defs
View File

@@ -1,7 +1,7 @@
def acceptance_tests( def acceptance_tests(
srcs, srcs,
deps = [], deps = [],
vm_args = []): vm_args = ['-Xmx128m']):
for j in srcs: for j in srcs:
java_test( java_test(
name = j[:-len('.java')], name = j[:-len('.java')],