Skip to content

Commit

Permalink
test: inherit JOBS from environment
Browse files Browse the repository at this point in the history
In some virtualized environments the amount of available resources
are misleading; for instance `multiprocessing.cpu_count()` on our current
4-core smartos vm's returns `48`. This is not a bug, merely how the vm
host provides information about available hardware.

Avoid running into issues by overriding `cpu_count()` with `JOBS`.

PR-URL: #4495
Reviewed-By: Brian White <mscdex@mscdex.net>
  • Loading branch information
jbergstroem authored and Fishrock123 committed Jan 6, 2016
1 parent 3bfc187 commit 08a3490
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion tools/test.py
Expand Up @@ -1339,7 +1339,10 @@ def ProcessOptions(options):
print "The test group to run (n) must be smaller than number of groups (m)."
return False
if options.J:
options.j = multiprocessing.cpu_count()
# inherit JOBS from environment if provided. some virtualised systems
# tends to exaggerate the number of available cpus/cores.
cores = os.environ.get('JOBS')
options.j = int(cores) if cores is not None else multiprocessing.cpu_count()
if options.flaky_tests not in ["run", "skip", "dontcare"]:
print "Unknown flaky-tests mode %s" % options.flaky_tests
return False
Expand Down

0 comments on commit 08a3490

Please sign in to comment.