valitydev/salt
14
0
Fork 0
mirror of https://github.com/valitydev/salt.git synced 2024-11-08 09:23:56 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Correct issue with docker api 1.19 due to move of mem parameters to host_config object

Browse Source
This commit is contained in:
nfraison 2015-08-26 15:04:56 +02:00
parent 809a47f299
commit 9f7ccc6645
2 changed files with 49 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

60
salt/modules/dockerio.py
View File

@ -660,25 +660,47 @@ def create_container(image,
volumes.remove(volume)
try:
container_info = client.create_container(
image=image,
command=command,
hostname=hostname,
user=user,
detach=detach,
stdin_open=stdin_open,
tty=tty,
mem_limit=mem_limit,
ports=ports,
environment=environment,
dns=dns,
volumes=volumes,
volumes_from=volumes_from,
name=name,
cpu_shares=cpu_shares,
cpuset=cpuset,
host_config=docker.utils.create_host_config(binds=binds)
)
if float(client.version()['ApiVersion']) > 1.18:
container_info = client.create_container(
image=image,
command=command,
hostname=hostname,
user=user,
detach=detach,
stdin_open=stdin_open,
tty=tty,
ports=ports,
environment=environment,
dns=dns,
volumes=volumes,
volumes_from=volumes_from,
name=name,
cpu_shares=cpu_shares,
cpuset=cpuset,
host_config=docker.utils.create_host_config(binds=binds,
mem_limit=mem_limit)
)
else:
container_info = client.create_container(
image=image,
command=command,
hostname=hostname,
user=user,
detach=detach,
stdin_open=stdin_open,
tty=tty,
mem_limit=mem_limit,
ports=ports,
environment=environment,
dns=dns,
volumes=volumes,
volumes_from=volumes_from,
name=name,
cpu_shares=cpu_shares,
cpuset=cpuset,
host_config=docker.utils.create_host_config(binds=binds)
)
log.trace("docker.client.create_container returned: " + str(container_info))
container = container_info['Id']
callback = _valid

8
salt/modules/dockerng.py
View File

@ -2658,6 +2658,14 @@ def create(image,
if 'api_name' in val:
create_kwargs[val['api_name']] = create_kwargs.pop(key)
# Added to manage api change in 1.19.
# mem_limit and memswap_limit must be provided in host_config object
if float(version()['ApiVersion']) > 1.18:
create_kwargs['host_config'] = docker.utils.create_host_config(mem_limit=create_kwargs.get('mem_limit'),
memswap_limit=create_kwargs.get('memswap_limit'))
if 'mem_limit' in create_kwargs: del create_kwargs['mem_limit']
if 'memswap_limit' in create_kwargs: del create_kwargs['memswap_limit']
log.debug(
'dockerng.create is using the following kwargs to create '
'container \'{0}\' from image \'{1}\': {2}'