From a23cd3d8e874635718e93b121dec58a08c8f766c Mon Sep 17 00:00:00 2001 From: Matthew Berry Date: Thu, 13 Jul 2017 23:20:24 -0500 Subject: [PATCH] Fix #1673 check resource error in container network API Container network functions checked 'image' as resource ID and not 'container'. This caused a traceback when using container as named argument. Signed-off-by: Matthew Berry --- docker/api/network.py | 4 ++-- tests/unit/api_network_test.py | 6 +++--- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/docker/api/network.py b/docker/api/network.py index 5ebb41af..5549bf0c 100644 --- a/docker/api/network.py +++ b/docker/api/network.py @@ -200,7 +200,7 @@ class NetworkApiMixin(object): res = self._get(url, params=params) return self._result(res, json=True) - @check_resource('image') + @check_resource('container') @minimum_version('1.21') def connect_container_to_network(self, container, net_id, ipv4_address=None, ipv6_address=None, @@ -237,7 +237,7 @@ class NetworkApiMixin(object): res = self._post_json(url, data=data) self._raise_for_status(res) - @check_resource('image') + @check_resource('container') @minimum_version('1.21') def disconnect_container_from_network(self, container, net_id, force=False): diff --git a/tests/unit/api_network_test.py b/tests/unit/api_network_test.py index f997a1b8..96cdc4b1 100644 --- a/tests/unit/api_network_test.py +++ b/tests/unit/api_network_test.py @@ -147,8 +147,8 @@ class NetworkTest(BaseAPIClientTest): with mock.patch('docker.api.client.APIClient.post', post): self.client.connect_container_to_network( - {'Id': container_id}, - network_id, + container={'Id': container_id}, + net_id=network_id, aliases=['foo', 'bar'], links=[('baz', 'quux')] ) @@ -176,7 +176,7 @@ class NetworkTest(BaseAPIClientTest): with mock.patch('docker.api.client.APIClient.post', post): self.client.disconnect_container_from_network( - {'Id': container_id}, network_id) + container={'Id': container_id}, net_id=network_id) self.assertEqual( post.call_args[0][0],