Merge pull request #732 from docker/version-dependent-hostconfig

Version dependent hostconfig

Author: aanand

docker/client.py

@@ -248,15 +248,18 @@ def create_container(self, image, command=None, hostname=None, user=None,
                 'host_config is not supported in API < 1.15'
             )
 
-        config = utils.create_container_config(
-            self._version, image, command, hostname, user, detach, stdin_open,
+        config = self.create_container_config(
+            image, command, hostname, user, detach, stdin_open,
             tty, mem_limit, ports, environment, dns, volumes, volumes_from,
             network_disabled, entrypoint, cpu_shares, working_dir, domainname,
             memswap_limit, cpuset, host_config, mac_address, labels,
             volume_driver
         )
         return self.create_container_from_config(config, name)
 
+    def create_container_config(self, *args, **kwargs):
+        return utils.create_container_config(self._version, *args, **kwargs)
+
     def create_container_from_config(self, config, name=None):
         u = self._url("/containers/create")
         params = {
@@ -265,6 +268,12 @@ def create_container_from_config(self, config, name=None):
         res = self._post_json(u, data=config, params=params)
         return self._result(res, True)
 
+    def create_host_config(self, *args, **kwargs):
+        if not kwargs:
+            kwargs = {}
+        kwargs['version'] = self._version
+        return utils.create_host_config(*args, **kwargs)
+
     @check_resource
     def diff(self, container):
         return self._result(self._get(self._url("/containers/{0}/changes".
@@ -815,7 +824,7 @@ def start(self, container, binds=None, port_bindings=None, lxc_conf=None,
                     'Please use host_config in create_container instead!',
                     DeprecationWarning
                 )
-            start_config = utils.create_host_config(**start_config_kwargs)
+            start_config = self.create_host_config(**start_config_kwargs)
 
         url = self._url("/containers/{0}/start".format(container))
         res = self._post_json(url, data=start_config)

docker/utils/utils.py

@@ -27,6 +27,7 @@
 import requests
 import six
 
+from .. import constants
 from .. import errors
 from .. import tls
 from .types import Ulimit, LogConfig
@@ -395,10 +396,17 @@ def create_host_config(
     restart_policy=None, cap_add=None, cap_drop=None, devices=None,
     extra_hosts=None, read_only=None, pid_mode=None, ipc_mode=None,
     security_opt=None, ulimits=None, log_config=None, mem_limit=None,
-    memswap_limit=None, cgroup_parent=None
+    memswap_limit=None, cgroup_parent=None, version=None
 ):
     host_config = {}
 
+    if not version:
+        warnings.warn(
+            'docker.utils.create_host_config() is deprecated. Please use '
+            'Client.create_host_config() instead.'
+        )
+        version = constants.DEFAULT_DOCKER_API_VERSION
+
     if mem_limit is not None:
         if isinstance(mem_limit, six.string_types):
             mem_limit = parse_bytes(mem_limit)
@@ -433,7 +441,7 @@ def create_host_config(
 
     if network_mode:
         host_config['NetworkMode'] = network_mode
-    elif network_mode is None:
+    elif network_mode is None and compare_version('1.19', version) > 0:
         host_config['NetworkMode'] = 'default'
 
     if restart_policy:

docs/api.md

@@ -234,7 +234,7 @@ from. Optionally a single string joining container id's with commas
  'Warnings': None}
 ```
 
-### parse_env_file
+### docker.utils.parse_env_file
 
 A utility for parsing an environment file.
 

docs/hostconfig.md

@@ -6,7 +6,7 @@ The Docker Remote API introduced [support for HostConfig in version 1.15](http:/
 
 ## HostConfig helper
 
-### docker.utils.create_host_config
+### Client.create_host_config
 
 Creates a HostConfig dictionary to be used with `Client.create_container`.
 
@@ -97,7 +97,8 @@ for example:
 **Returns** (dict) HostConfig dictionary
 
 ```python
->>> from docker.utils import create_host_config
->>> create_host_config(privileged=True, cap_drop=['MKNOD'], volumes_from=['nostalgic_newton'])
+>>> from docker import Client
+>>> c = Client()
+>>> c.create_host_config(privileged=True, cap_drop=['MKNOD'], volumes_from=['nostalgic_newton'])
 {'CapDrop': ['MKNOD'], 'LxcConf': None, 'Privileged': True, 'VolumesFrom': ['nostalgic_newton'], 'PublishAllPorts': False}
 ```