openstack/python-glanceclient
Code Issues Proposed changes

Merge "Log request-id before exceptions raised"

Browse Source
This commit is contained in:
Jenkins
2016-07-30 22:01:48 +00:00
committed by Gerrit Code Review
parent ee7e22fa52 155183a21e
commit c039237007
1 changed files with 10 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
glanceclient/common/http.py
View File

@@ -92,6 +92,16 @@ class _BaseHTTPClient(object):
return data return data
def _handle_response(self, resp): def _handle_response(self, resp):
# log request-id for each api cal
request_id = resp.headers.get('x-openstack-request-id')
if request_id:
LOG.debug('%(method)s call to glance-api for '
'%(url)s used request id '
'%(response_request_id)s',
{'method': resp.request.method,
'url': resp.url,
'response_request_id': request_id})
if not resp.ok: if not resp.ok:
LOG.debug("Request returned failure status %s." % resp.status_code) LOG.debug("Request returned failure status %s." % resp.status_code)
raise exc.from_response(resp, resp.content) raise exc.from_response(resp, resp.content)
@@ -120,16 +130,6 @@ class _BaseHTTPClient(object):
except ValueError: except ValueError:
body_iter = None body_iter = None
# log request-id for each api call
request_id = resp.headers.get('x-openstack-request-id')
if request_id:
LOG.debug('%(method)s call to glance-api for '
'%(url)s used request id '
'%(response_request_id)s',
{'method': resp.request.method,
'url': resp.url,
'response_request_id': request_id})
return resp, body_iter return resp, body_iter