Skip to content

Commit

Permalink
Fix Windows refleak timeout
Browse files Browse the repository at this point in the history
* Set the test_timeout on Windows64RefleakBuild
* WindowsBuild now uses regrtest --timeout=xxx even if the
  test_timeout class variable is not set: use the default timeout in
  that case
  • Loading branch information
vstinner committed Jun 15, 2017
1 parent 3b8215d commit cb5a907
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions master/master.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -316,11 +316,11 @@ class WindowsBuild(factory.BuildFactory):
self.addStep(Compile(command=build_command))
# timeout is a bit more than the regrtest default timeout
if self.test_timeout:
if branch != '2.7':
test_command += ['--timeout', self.test_timeout - (5 * 60)]
timeout = self.test_timeout
else:
timeout = TEST_TIMEOUT
if branch != '2.7':
test_command += ['--timeout', timeout - (5 * 60)]
self.addStep(Test(command=test_command, timeout=timeout))
self.addStep(Clean(command=clean_command))

Expand Down Expand Up @@ -356,6 +356,10 @@ class Windows64Build(WindowsBuild):
class Windows64RefleakBuild(Windows64Build):
buildersuffix = '.refleak'
testFlags = ['-x64', '-j2', '-R', '3:3:refleaks.log', '-u-cpu']
# -R 3:3 is supposed to only require timeout x 6, but in practice,
# it's much more slower. Use timeout x 10 to prevent timeout
# caused by --huntrleaks.
test_timeout = TEST_TIMEOUT * 10


class Windows64ReleaseBuild(Windows64Build):
Expand Down

0 comments on commit cb5a907

Please sign in to comment.