mirror of
https://github.com/docker/compose.git
synced 2025-07-23 13:45:00 +02:00
Reduce up() cyclomatic complexity
Signed-off-by: Joffrey F <joffrey@docker.com>
This commit is contained in:
parent
376389d7a5
commit
3cb22fa94a
@ -969,7 +969,31 @@ class TopLevelCommand(object):
|
||||
|
||||
if cascade_stop:
|
||||
print("Aborting on container exit...")
|
||||
all_containers = self.project.containers(service_names=options['SERVICE'], stopped=True)
|
||||
exit_code = compute_exit_code(
|
||||
exit_value_from, attached_containers, cascade_starter, all_containers
|
||||
)
|
||||
|
||||
self.project.stop(service_names=service_names, timeout=timeout)
|
||||
sys.exit(exit_code)
|
||||
|
||||
@classmethod
|
||||
def version(cls, options):
|
||||
"""
|
||||
Show version informations
|
||||
|
||||
Usage: version [--short]
|
||||
|
||||
Options:
|
||||
--short Shows only Compose's version number.
|
||||
"""
|
||||
if options['--short']:
|
||||
print(__version__)
|
||||
else:
|
||||
print(get_version_info('full'))
|
||||
|
||||
|
||||
def compute_exit_code(exit_value_from, attached_containers, cascade_starter, all_containers):
|
||||
exit_code = 0
|
||||
if exit_value_from:
|
||||
candidates = list(filter(
|
||||
@ -991,29 +1015,13 @@ class TopLevelCommand(object):
|
||||
else:
|
||||
exit_code = candidates[0].inspect()['State']['ExitCode']
|
||||
else:
|
||||
for e in self.project.containers(service_names=options['SERVICE'], stopped=True):
|
||||
for e in all_containers:
|
||||
if (not e.is_running and cascade_starter == e.name):
|
||||
if not e.exit_code == 0:
|
||||
exit_code = e.exit_code
|
||||
break
|
||||
|
||||
self.project.stop(service_names=service_names, timeout=timeout)
|
||||
sys.exit(exit_code)
|
||||
|
||||
@classmethod
|
||||
def version(cls, options):
|
||||
"""
|
||||
Show version informations
|
||||
|
||||
Usage: version [--short]
|
||||
|
||||
Options:
|
||||
--short Shows only Compose's version number.
|
||||
"""
|
||||
if options['--short']:
|
||||
print(__version__)
|
||||
else:
|
||||
print(get_version_info('full'))
|
||||
return exit_code
|
||||
|
||||
|
||||
def convergence_strategy_from_opts(options):
|
||||
|
Loading…
x
Reference in New Issue
Block a user