diff options
author | Lee Yarwood <lyarwood@redhat.com> | 2020-12-18 11:14:51 +0000 |
---|---|---|
committer | Jay Bryant <jungleboyj@electronicjungle.net> | 2021-03-10 16:31:49 +0000 |
commit | 0439a44707788f9400e5aba716001f4e4bb55d81 (patch) | |
tree | 18ff90c244651250e5c5e0ace0f783965726e2f2 | |
parent | 617fdb9ca64b140b145fa2a1c6ca13e27d6fea63 (diff) | |
download | python-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.py | 14 |
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 |