Move rename functionality into Container

Signed-off-by: Mazz Mosley <mazz@houseofmnowster.com>
This commit is contained in:
Mazz Mosley 2015-10-27 11:55:35 +00:00
parent bee063c07d
commit 30a84f1be6
2 changed files with 10 additions and 8 deletions

View File

@ -192,6 +192,15 @@ class Container(object):
def remove(self, **options):
return self.client.remove_container(self.id, **options)
def rename_to_tmp_name(self):
"""Rename the container to a hopefully unique temporary container name
by prepending the short id.
"""
self.client.rename(
self.id,
'%s_%s' % (self.short_id, self.name)
)
def inspect_if_not_inspected(self):
if not self.has_been_inspected:
self.inspect()

View File

@ -446,13 +446,6 @@ class Service(object):
else:
raise
def _recreate_rename_container(self, container):
# Use a hopefully unique container name by prepending the short id
self.client.rename(
container.id,
'%s_%s' % (container.short_id, container.name)
)
def recreate_container(self,
container,
timeout=DEFAULT_TIMEOUT,
@ -466,7 +459,7 @@ class Service(object):
log.info("Recreating %s" % container.name)
self._recreate_stop_container(container, timeout)
self._recreate_rename_container(container)
container.rename_to_tmp_name()
new_container = self.create_container(
do_build=False,
previous_container=container,