jyapayne/compose
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Remove service.start_container()

Browse source 
It has been an unnecessary wrapper around container.start() for a little while now, so we can call it directly.

Signed-off-by: Daniel Nephin <dnephin@docker.com>
This commit is contained in:
Daniel Nephin 2015-11-02 13:33:13 -05:00
commit e317d2db9d
5 changed files with 22 additions and 26 deletions
Download patch file Download diff file Expand all files Collapse all files

13
compose/service.py
View file

@ -410,7 +410,7 @@ class Service(object):
if should_attach_logs:
container.attach_log_stream()
self.start_container(container)
container.start()
return [container]
@ -464,21 +464,16 @@ class Service(object):
)
if attach_logs:
new_container.attach_log_stream()
self.start_container(new_container)
new_container.start()
container.remove()
return new_container
def start_container_if_stopped(self, container, attach_logs=False):
if container.is_running:
return container
else:
if not container.is_running:
log.info("Starting %s" % container.name)
if attach_logs:
container.attach_log_stream()
return self.start_container(container)
def start_container(self, container):
container.start()
container.start()
return container
def remove_duplicate_containers(self, timeout=DEFAULT_TIMEOUT):