summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLee Yarwood <lyarwood@redhat.com>2020-12-18 11:14:51 +0000
committerJay Bryant <jungleboyj@electronicjungle.net>2021-03-10 16:31:49 +0000
commit0439a44707788f9400e5aba716001f4e4bb55d81 (patch)
tree18ff90c244651250e5c5e0ace0f783965726e2f2
parent617fdb9ca64b140b145fa2a1c6ca13e27d6fea63 (diff)
downloadpython-cinderclient-0439a44707788f9400e5aba716001f4e4bb55d81.tar.gz
client: Stop logging request-id twice in DEBUG
This is already logged by http_log_resp in DEBUG along with the other response headers so drop the duplicate logging in _log_request_id. Change-Id: I3c5ad2b480d80611ecb10449068d836c4bbe7bdb
-rw-r--r--cinderclient/client.py14
1 files changed, 0 insertions, 14 deletions
diff --git a/cinderclient/client.py b/cinderclient/client.py
index 2d7b80d..2fed6fb 100644
--- a/cinderclient/client.py
+++ b/cinderclient/client.py
@@ -155,16 +155,6 @@ def get_volume_api_from_url(url):
raise exceptions.UnsupportedVersion(msg)
-def _log_request_id(logger, resp, service_name):
- request_id = resp.headers.get('x-openstack-request-id')
- if request_id:
- logger.debug('%(method)s call to %(service_type)s for %(url)s '
- 'used request id %(response_request_id)s',
- {'method': resp.request.method,
- 'service_type': service_name,
- 'url': resp.url, 'response_request_id': request_id})
-
-
class SessionClient(adapter.LegacyJsonAdapter):
def __init__(self, *args, **kwargs):
@@ -378,10 +368,6 @@ class HTTPClient(object):
resp.headers,
strutils.mask_password(resp.text))
- # if service name is None then use service_type for logging
- service = self.service_name or self.service_type
- _log_request_id(self._logger, resp, service)
-
def request(self, url, method, **kwargs):
kwargs.setdefault('headers', kwargs.get('headers', {}))
kwargs['headers']['User-Agent'] = self.USER_AGENT