summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJenkins <jenkins@review.openstack.org>2017-01-24 20:35:49 +0000
committerGerrit Code Review <review@openstack.org>2017-01-24 20:35:49 +0000
commit8cd3e87fc9378d36954ae7505d207ba9b44c53b9 (patch)
tree86f160f18dad14eadf5d14f5d1827315a7f11f47
parentfe6803c88b8bcb58ab609c777ec0215ea1e4f3a8 (diff)
parent6962056c67dca1fee2e832ff8bfbc317ed4b5ec0 (diff)
downloadpython-cinderclient-8cd3e87fc9378d36954ae7505d207ba9b44c53b9.tar.gz
Merge "x-openstack-request-id logged twice in logs"
-rw-r--r--cinderclient/client.py4
-rw-r--r--cinderclient/tests/unit/test_client.py15
2 files changed, 3 insertions, 16 deletions
diff --git a/cinderclient/client.py b/cinderclient/client.py
index f503e83..43157e6 100644
--- a/cinderclient/client.py
+++ b/cinderclient/client.py
@@ -115,10 +115,6 @@ class SessionClient(adapter.LegacyJsonAdapter):
raise_exc=False,
**kwargs)
- # 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)
-
if raise_exc and resp.status_code >= 400:
raise exceptions.from_response(resp, body)
diff --git a/cinderclient/tests/unit/test_client.py b/cinderclient/tests/unit/test_client.py
index e13a772..bc7d914 100644
--- a/cinderclient/tests/unit/test_client.py
+++ b/cinderclient/tests/unit/test_client.py
@@ -101,11 +101,10 @@ class ClientTest(utils.TestCase):
cs = cinderclient.client.SessionClient(self, api_version='3.0')
self.assertEqual('http://192.168.122.104:8776/', cs._get_base_url())
- @mock.patch.object(cinderclient.client, '_log_request_id')
@mock.patch.object(adapter.Adapter, 'request')
@mock.patch.object(exceptions, 'from_response')
def test_sessionclient_request_method(
- self, mock_from_resp, mock_request, mock_log):
+ self, mock_from_resp, mock_request):
kwargs = {
"body": {
"volume": {
@@ -139,18 +138,15 @@ class ClientTest(utils.TestCase):
response, body = session_client.request(mock.sentinel.url,
'POST', **kwargs)
self.assertIsNotNone(session_client._logger)
- mock_log.assert_called_once_with(session_client._logger, mock_response,
- mock.ANY)
# In this case, from_response method will not get called
# because response status_code is < 400
self.assertEqual(202, response.status_code)
self.assertFalse(mock_from_resp.called)
- @mock.patch.object(cinderclient.client, '_log_request_id')
@mock.patch.object(adapter.Adapter, 'request')
def test_sessionclient_request_method_raises_badrequest(
- self, mock_request, mock_log):
+ self, mock_request):
kwargs = {
"body": {
"volume": {
@@ -185,13 +181,10 @@ class ClientTest(utils.TestCase):
self.assertRaises(exceptions.BadRequest, session_client.request,
mock.sentinel.url, 'POST', **kwargs)
self.assertIsNotNone(session_client._logger)
- mock_log.assert_called_once_with(session_client._logger, mock_response,
- mock.ANY)
- @mock.patch.object(cinderclient.client, '_log_request_id')
@mock.patch.object(adapter.Adapter, 'request')
def test_sessionclient_request_method_raises_overlimit(
- self, mock_request, mock_log):
+ self, mock_request):
resp = {
"overLimitFault": {
"message": "This request was rate-limited.",
@@ -212,8 +205,6 @@ class ClientTest(utils.TestCase):
self.assertRaises(exceptions.OverLimit, session_client.request,
mock.sentinel.url, 'GET')
self.assertIsNotNone(session_client._logger)
- mock_log.assert_called_once_with(session_client._logger, mock_response,
- mock.ANY)
@mock.patch.object(exceptions, 'from_response')
def test_keystone_request_raises_auth_failure_exception(