[PATCH 12/19] PerfRepo: add log messages to PerfRepoRESTAPI

[Ondrej Lichtner]

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

Basic log messages to improve usability.

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

diff --git a/lnst/Controller/PerfRepo.py b/lnst/Controller/PerfRepo.py
index f05884a..ba3fd64 100644
--- a/lnst/Controller/PerfRepo.py
+++ b/lnst/Controller/PerfRepo.py
@@ -818,6 +818,7 @@ class PerfRepoRESTAPI(object):
             logging.debug(response.text)
             return None
         else:
+            logging.info("GET %s success" % get_url)
             return PerfRepoTest(response.content)
 
     def test_get_by_uid(self, test_uid):
@@ -827,6 +828,7 @@ class PerfRepoRESTAPI(object):
             logging.debug(response.text)
             return None
         else:
+            logging.info("GET %s success" % get_url)
             return PerfRepoTest(response.content)
 
     def test_create(self, test):
@@ -838,6 +840,8 @@ class PerfRepoRESTAPI(object):
         else:
             new_id = response.headers["Location"].split('/')[-1]
             test.set_id(new_id)
+            logging.info("POST %s success" % post_url)
+            logging.info("Obj url: %s" % self.get_obj_url(test))
             return test
 
     def test_add_metric(self, test_id, metric):
@@ -849,6 +853,8 @@ class PerfRepoRESTAPI(object):
         else:
             new_id = response.headers["Location"].split('/')[-1]
             metric.set_id(new_id)
+            logging.info("POST %s success" % post_url)
+            logging.info("Obj url: %s" % self.get_obj_url(test))
             return metric
 
     def test_remove(self, test_id):
@@ -857,6 +863,7 @@ class PerfRepoRESTAPI(object):
         if response.status_code != 204:
             return False
         else:
+            logging.info("DELETE %s success" % delete_url)
             return True
 
     def metric_get(self, metric_id):
@@ -866,6 +873,7 @@ class PerfRepoRESTAPI(object):
             logging.debug(response.text)
             return None
         else:
+            logging.info("GET %s success" % get_url)
             return PerfRepoMetric(response.content)
 
     def testExecution_get(self, testExec_id):
@@ -875,6 +883,7 @@ class PerfRepoRESTAPI(object):
             logging.debug(response.text)
             return None
         else:
+            logging.info("GET %s success" % get_url)
             return PerfRepoTestExecution(response.content)
 
     def testExecution_create(self, testExec):
@@ -886,6 +895,8 @@ class PerfRepoRESTAPI(object):
         else:
             new_id = response.headers["Location"].split('/')[-1]
             testExec.set_id(new_id)
+            logging.info("POST %s success" % post_url)
+            logging.info("Obj url: %s" % self.get_obj_url(testExec))
             return testExec
 
     def testExecution_delete(self, testExec_id):
@@ -895,6 +906,7 @@ class PerfRepoRESTAPI(object):
             logging.debug(response.text)
             return False
         else:
+            logging.info("DELETE %s success" % delete_url)
             return True
 
     def testExecution_add_value(self, value):
@@ -921,6 +933,7 @@ class PerfRepoRESTAPI(object):
             logging.debug(response.text)
             return None
         else:
+            logging.info("GET %s success" % get_url)
             return PerfRepoReport(response.content)
 
     def report_create(self, report):
@@ -935,6 +948,8 @@ class PerfRepoRESTAPI(object):
         else:
             new_id = response.headers["Location"].split('/')[-1]
             report.set_id(new_id)
+            logging.info("POST %s success" % post_url)
+            logging.info("Obj url: %s" % self.get_obj_url(report))
             return report
 
     def report_update(self, report):
@@ -947,6 +962,8 @@ class PerfRepoRESTAPI(object):
             logging.debug(response.text)
             return None
         else:
+            logging.info("UPDATE %s success" % post_url)
+            logging.info("Obj url: %s" % self.get_obj_url(report))
             return report
 
     def report_delete(self, report_id):
-- 
2.5.2