fix remove_orphans using APIs exposed via AnsibleDockerClient (#54316)

Co-Authored-By: sluther <neenach2002@gmail.com>

(cherry picked from commit 5517b0384f)
pull/54918/head
Scott Luther 6 years ago committed by Toshio Kuratomi
parent 3af578ff20
commit 78ebe74b61

@ -0,0 +1,2 @@
bugfixes:
- docker_service - fixed an issue where ``remove_orphans`` doesn't work reliably.

@ -457,7 +457,7 @@ try:
from compose.cli.command import project_from_options from compose.cli.command import project_from_options
from compose.service import NoSuchImageError from compose.service import NoSuchImageError
from compose.cli.main import convergence_strategy_from_opts, build_action_from_opts, image_type_from_opt from compose.cli.main import convergence_strategy_from_opts, build_action_from_opts, image_type_from_opt
from compose.const import DEFAULT_TIMEOUT from compose.const import DEFAULT_TIMEOUT, LABEL_SERVICE, LABEL_PROJECT, LABEL_ONE_OFF
HAS_COMPOSE = True HAS_COMPOSE = True
HAS_COMPOSE_EXC = None HAS_COMPOSE_EXC = None
MINIMUM_COMPOSE_VERSION = '1.7.0' MINIMUM_COMPOSE_VERSION = '1.7.0'
@ -711,6 +711,25 @@ class ContainerManager(DockerBaseClass):
result['changed'] = build_output['changed'] result['changed'] = build_output['changed']
result['actions'] += build_output['actions'] result['actions'] += build_output['actions']
if self.remove_orphans:
containers = self.client.containers(
filters={
'label': [
'{0}={1}'.format(LABEL_PROJECT, self.project.name),
'{0}={1}'.format(LABEL_ONE_OFF, "False")
],
}
)
orphans = []
for container in containers:
service_name = container.get('Labels', {}).get(LABEL_SERVICE)
if service_name not in self.project.service_names:
orphans.append(service_name)
if orphans:
result['changed'] = True
for service in self.project.services: for service in self.project.services:
if not service_names or service.name in service_names: if not service_names or service.name in service_names:
plan = service.convergence_plan(strategy=converge) plan = service.convergence_plan(strategy=converge)

Loading…
Cancel
Save