commit 26e2427b80a129b9e8cd95ba7d24c633081ca884
author Jenkins <jenkins@review.openstack.org> Thu Oct 23 16:55:50 2014 +0000
committer Gerrit Code Review <review@openstack.org> Thu Oct 23 16:55:51 2014 +0000
tree 09ea101ccbb2b2a9912be155f1986732013bba4c
parent e0befa487d99da7e7b79b0488ffac6a86e055ac0
parent 75d2e62442ef4d05249be1ff481356905f41ade8

Merge "Don't double log API request"

tempest/common/rest_client.py

diff --git a/tempest/common/rest_client.py b/tempest/common/rest_client.py
index 42e4f56..3a0e975 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.