commit 75d2e62442ef4d05249be1ff481356905f41ade8
author Joe Gordon <joe.gordon0@gmail.com> Tue Oct 14 12:31:22 2014 -0700
committer Joe Gordon <joe.gordon0@gmail.com> Wed Oct 15 11:30:44 2014 -0700
tree 655caf3b8f01e44c3a6da0af2fa8d09e56f75d55
parent 41d6aa233bf9c76734d443e023bb88a448f6ed59

Don't double log API request

If debug logging is enabled the full API call, request and response is
logged, so don't log the request twice.

Change-Id: I45baa04a2daae8fd31c30183ac10464b7affbc21

tempest/common/rest_client.py

diff --git a/tempest/common/rest_client.py b/tempest/common/rest_client.py
index 00fe8d2..8227c89 100644
--- a/tempest/common/rest_client.py
+++ b/tempest/common/rest_client.py
@@ -16,6 +16,7 @@
 
 import collections
 import json
+import logging as real_logging
 import re
 import time
 
@@ -310,14 +311,15 @@
         caller_name = misc_utils.find_test_caller()
         if secs:
             secs = " %.3fs" % secs
-        self.LOG.info(
-            'Request (%s): %s %s %s%s' % (
-                caller_name,
-                resp['status'],
-                method,
-                req_url,
-                secs),
-            extra=extra)
+        if not self.LOG.isEnabledFor(real_logging.DEBUG):
+            self.LOG.info(
+                'Request (%s): %s %s %s%s' % (
+                    caller_name,
+                    resp['status'],
+                    method,
+                    req_url,
+                    secs),
+                extra=extra)
 
         # Also look everything at DEBUG if you want to filter this
         # out, don't run at debug.