From 0ef288dcc827fae9ab2db137a9434717fa561250 Mon Sep 17 00:00:00 2001 From: Joshua Conner Date: Mon, 14 Apr 2014 15:05:42 -0700 Subject: [PATCH] Docker: don't send wrong parameters to client.start() As part of being updated for the 1.10 API, a couple of parameters were passed to the docker.client.start() command that it doesn't accept. This caused the module to error out if it tried to start any Docker containers. This removes those parameters so the module works again. --- library/cloud/docker | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/library/cloud/docker b/library/cloud/docker index 3fb82fd7dc5..8288d366b6a 100644 --- a/library/cloud/docker +++ b/library/cloud/docker @@ -566,10 +566,6 @@ class DockerManager: 'links': self.links, } - if docker.utils.compare_version('1.10', self.client.version()['ApiVersion']) >= 0: - params['dns'] = self.module.params.get('dns') - params['volumes_from'] = self.module.params.get('volumes_from') - for i in containers: self.client.start(i['Id'], **params) self.increment_counter('started') @@ -677,11 +673,11 @@ def main(): # start/stop containers if state in [ "running", "present" ]: - + # make sure a container with `name` exists, if not create and start it if name and "/" + name not in map(lambda x: x.get('Name'), deployed_containers): containers = manager.create_containers(1) - if state == "present": #otherwise it get (re)started later anyways.. + if state == "present": #otherwise it get (re)started later anyways.. manager.start_containers(containers) running_containers = manager.get_running_containers() deployed_containers = manager.get_deployed_containers() @@ -690,18 +686,18 @@ def main(): # make sure a container with `name` is running if name and "/" + name not in map(lambda x: x.get('Name'), running_containers): manager.start_containers(deployed_containers) - + # start more containers if we don't have enough elif delta > 0: containers = manager.create_containers(delta) manager.start_containers(containers) - + # stop containers if we have too many elif delta < 0: containers_to_stop = running_containers[0:abs(delta)] containers = manager.stop_containers(containers_to_stop) manager.remove_containers(containers_to_stop) - + facts = manager.get_running_containers() else: acts = manager.get_deployed_containers()