Merge pull request #1704 from aanand/fix-timeout-type

Make sure up/restart/stop timeout is an int
This commit is contained in:
Mazz Mosley 2015-07-16 10:50:06 +01:00
commit c7dccccd1f
2 changed files with 5 additions and 5 deletions

View File

@ -405,7 +405,7 @@ class TopLevelCommand(Command):
-t, --timeout TIMEOUT Specify a shutdown timeout in seconds.
(default: 10)
"""
timeout = float(options.get('--timeout') or DEFAULT_TIMEOUT)
timeout = int(options.get('--timeout') or DEFAULT_TIMEOUT)
project.stop(service_names=options['SERVICE'], timeout=timeout)
def restart(self, project, options):
@ -418,7 +418,7 @@ class TopLevelCommand(Command):
-t, --timeout TIMEOUT Specify a shutdown timeout in seconds.
(default: 10)
"""
timeout = float(options.get('--timeout') or DEFAULT_TIMEOUT)
timeout = int(options.get('--timeout') or DEFAULT_TIMEOUT)
project.restart(service_names=options['SERVICE'], timeout=timeout)
def up(self, project, options):
@ -461,7 +461,7 @@ class TopLevelCommand(Command):
allow_recreate = not options['--no-recreate']
smart_recreate = options['--x-smart-recreate']
service_names = options['SERVICE']
timeout = float(options.get('--timeout') or DEFAULT_TIMEOUT)
timeout = int(options.get('--timeout') or DEFAULT_TIMEOUT)
to_attach = project.up(
service_names=service_names,

View File

@ -13,7 +13,7 @@ from .testcases import DockerClientTestCase
class ProjectTestCase(DockerClientTestCase):
def run_up(self, cfg, **kwargs):
kwargs.setdefault('smart_recreate', True)
kwargs.setdefault('timeout', 0.1)
kwargs.setdefault('timeout', 1)
project = self.make_project(cfg)
project.up(**kwargs)
@ -171,7 +171,7 @@ def converge(service,
plan,
insecure_registry=insecure_registry,
do_build=do_build,
timeout=0.1,
timeout=1,
)