]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Dell SC: Adding logging to httpclient
authorTom Swanson <tom_swanson@dell.com>
Tue, 12 Jan 2016 19:28:45 +0000 (13:28 -0600)
committerTom Swanson <tom_swanson@dell.com>
Tue, 12 Jan 2016 23:37:14 +0000 (17:37 -0600)
Added url and payload logging to REST calls in the
HttpClient class of the Dell Storage Center Cinder
driver.

Change-Id: Iedf915a8678698721e05a938712339e7715b333e

cinder/volume/drivers/dell/dell_storagecenter_api.py

index abd8291a8e520f6937c4308328fde467b159f188..6a438c8e54ccb96edce490e160f5c30b7f21c92c 100644 (file)
@@ -112,6 +112,7 @@ class HttpClient(object):
 
     @utils.retry(exceptions=(requests.ConnectionError,))
     def get(self, url):
+        LOG.debug('get: %(url)s', {'url': url})
         return self.session.get(
             self.__formatUrl(url),
             headers=self.header,
@@ -119,6 +120,9 @@ class HttpClient(object):
 
     @utils.retry(exceptions=(requests.ConnectionError,))
     def post(self, url, payload):
+        LOG.debug('post: %(url)s data: %(payload)s',
+                  {'url': url,
+                   'payload': payload})
         return self.session.post(
             self.__formatUrl(url),
             data=json.dumps(payload,
@@ -128,6 +132,9 @@ class HttpClient(object):
 
     @utils.retry(exceptions=(requests.ConnectionError,))
     def put(self, url, payload):
+        LOG.debug('put: %(url)s data: %(payload)s',
+                  {'url': url,
+                   'payload': payload})
         return self.session.put(
             self.__formatUrl(url),
             data=json.dumps(payload,
@@ -137,6 +144,7 @@ class HttpClient(object):
 
     @utils.retry(exceptions=(requests.ConnectionError,))
     def delete(self, url):
+        LOG.debug('delete: %(url)s', {'url': url})
         return self.session.delete(
             self.__formatUrl(url),
             headers=self.header,