diff options
| author | Joffrey F <joffrey@docker.com> | 2017-04-10 16:02:43 -0700 |
|---|---|---|
| committer | Joffrey F <joffrey@docker.com> | 2017-04-10 16:03:13 -0700 |
| commit | e506a2b8eaef4cea0453af0f40870e488a66d568 (patch) | |
| tree | ee07ff31e2bc565916e47fbb8dfa7d660c40e84b | |
| parent | abd5370d8750dd4472dfef3b51751ec7ffc1ed2c (diff) | |
| download | docker-py-e506a2b8eaef4cea0453af0f40870e488a66d568.tar.gz | |
Standardize handling of verbose param in inspect_network
Signed-off-by: Joffrey F <joffrey@docker.com>
| -rw-r--r-- | docker/api/network.py | 21 |
1 files changed, 10 insertions, 11 deletions
diff --git a/docker/api/network.py b/docker/api/network.py index 46cd68c..74f4cd2 100644 --- a/docker/api/network.py +++ b/docker/api/network.py @@ -168,24 +168,23 @@ class NetworkApiMixin(object): self._raise_for_status(res) @minimum_version('1.21') - def inspect_network(self, net_id, verbose=False): + def inspect_network(self, net_id, verbose=None): """ Get detailed information about a network. Args: net_id (str): ID of network - verbose (bool): If set shows the service details across the cluster - in swarm mode + verbose (bool): Show the service details across the cluster in + swarm mode. """ - if verbose is True: + params = {} + if verbose is not None: if version_lt(self._version, '1.28'): - raise InvalidVersion( - 'Verbose option was introduced in API 1.28' - ) - url = self._url("/networks/{0}?verbose=true", net_id) - else: - url = self._url("/networks/{0}", net_id) - res = self._get(url) + raise InvalidVersion('verbose was introduced in API 1.28') + params['verbose'] = verbose + + url = self._url("/networks/{0}", net_id) + res = self._get(url, params=params) return self._result(res, json=True) @check_resource |
