[PATCH 11/12] PerfRepo: log http response on failed request

[Ondrej Lichtner]

From: Ondrej Lichtner <olichtne(a)redhat.com&gt;

If the response code of a request is not success we should at least log
the contents of the returned message for debugging purposes.
Unfortunately these are just blocks of unformatted html, but it's better
than nothing.

Signed-off-by: Ondrej Lichtner <olichtne(a)redhat.com&gt;
---
 lnst/Controller/PerfRepo.py | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/lnst/Controller/PerfRepo.py b/lnst/Controller/PerfRepo.py
index 0ee5eeb..4bcbc56 100644
--- a/lnst/Controller/PerfRepo.py
+++ b/lnst/Controller/PerfRepo.py
@@ -14,6 +14,7 @@ import requests
 import collections
 import datetime
 import re
+import logging
 from types import StringType, NoneType
 from xml.etree import ElementTree
 from xml.etree.ElementTree import Element
@@ -432,6 +433,7 @@ class PerfRepoRESTAPI(object):
         get_url = self._url + '/rest/test/id/%s' % test_id
         response = self._session.get(get_url)
         if response.status_code != 200:
+            logging.debug(response.text)
             return None
         else:
             return PerfRepoTest(response.content)
@@ -440,6 +442,7 @@ class PerfRepoRESTAPI(object):
         get_url = self._url + '/rest/test/uid/%s' % test_uid
         response = self._session.get(get_url)
         if response.status_code != 200:
+            logging.debug(response.text)
             return None
         else:
             return PerfRepoTest(response.content)
@@ -448,6 +451,7 @@ class PerfRepoRESTAPI(object):
         post_url = self._url + '/rest/test/create'
         response = self._session.post(post_url, data=test.to_xml_string())
         if response.status_code != 201:
+            logging.debug(response.text)
             return None
         else:
             test.set_id(response.headers['Location'])
@@ -457,6 +461,7 @@ class PerfRepoRESTAPI(object):
         post_url = self._url + '/rest/test/id/%s/addMetric' % test_id
         response = self._session.post(post_url, data=metric.to_xml_string)
         if response.status_code != 201:
+            logging.debug(response.text)
             return None
         else:
             metric.set_id(response.headers['Location'])
@@ -474,6 +479,7 @@ class PerfRepoRESTAPI(object):
         get_url = self._url + '/rest/metric/%s' % metric_id
         response = self._session.get(get_url)
         if response.status_code != 200:
+            logging.debug(response.text)
             return None
         else:
             return PerfRepoMetric(response.content)
@@ -482,6 +488,7 @@ class PerfRepoRESTAPI(object):
         get_url = self._url + '/rest/testExecution/%s' % testExec_id
         response = self._session.get(get_url)
         if response.status_code != 200:
+            logging.debug(response.text)
             return None
         else:
             return PerfRepoTestExecution(response.content)
@@ -490,6 +497,7 @@ class PerfRepoRESTAPI(object):
         post_url = self._url + '/rest/testExecution/create'
         response = self._session.post(post_url, data=testExec.to_xml_string())
         if response.status_code != 201:
+            logging.debug(response.text)
             return None
         else:
             testExec.set_id(response.headers['Location'])
@@ -499,6 +507,7 @@ class PerfRepoRESTAPI(object):
         delete_url = self._url + '/rest/testExecution/%s' % testExec_id
         response = self._session.delete(delete_url)
         if response.status_code != 204:
+            logging.debug(response.text)
             return False
         else:
             return True
@@ -524,6 +533,7 @@ class PerfRepoRESTAPI(object):
         get_url = self._url + '/rest/report/id/%s' % report_id
         response = self._session.get(get_url)
         if response.status_code != 200:
+            logging.debug(response.text)
             return None
         else:
             return PerfRepoReport(response.content)
-- 
2.1.0