Update the logging namespace - openvswitch

Issue: autotest#3048 Signed-off-by: Xu Han <[email protected]>
leidwang · Aug 5, 2022 · a859290 · a859290
1 parent d552615
commit a859290
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 12 deletions.
diff --git a/openvswitch/tests/load_module.py b/openvswitch/tests/load_module.py
@@ -1,6 +1,5 @@
 import sys
 import traceback
-import logging
 import six
 
 from avocado.utils import process
@@ -44,8 +43,8 @@ def run(test, params, env):
             if _e is None:
                 raise
             else:
-                logging.error("Cleaning function raised exception too: \n%s",
-                              "".join(traceback.format_exception(e[0],
-                                                                 e[1],
-                                                                 e[2])))
+                test.log.error("Cleaning function raised exception too: \n%s",
+                               "".join(traceback.format_exception(e[0],
+                                                                  e[1],
+                                                                  e[2])))
                 six.reraise(_e[0], _e[1], _e[2])
diff --git a/openvswitch/tests/ovs_basic.py b/openvswitch/tests/ovs_basic.py
@@ -13,6 +13,8 @@
 from virttest import data_dir
 from virttest import error_context
 
+LOG_JOB = logging.getLogger('avocado.test')
+
 
 def allow_iperf_firewall(machine):
     machine.cmd("iptables -I INPUT -p tcp --dport 5001 --j ACCEPT")
@@ -70,7 +72,7 @@ def setup(self, test, params, env):
                                           self.br0_name,
                                           self.ovs)
 
-        logging.debug(self.ovs.status())
+        LOG_JOB.debug(self.ovs.status())
         self.host = ovs_utils.Machine(src=test.workdir)
         self.mvms = [ovs_utils.Machine(vm) for vm in self.vms]
         self.machines = [self.host] + self.mvms
@@ -174,16 +176,16 @@ def test(self, test, params, env):
             # Test TCP bandwidth
             error_context.context("Test iperf bandwidth tcp.")
             speeds = self.test_bandwidth()
-            logging.info("TCP Bandwidth from vm->host: %s", speeds[0])
-            logging.info("TCP Bandwidth from host->vm: %s", speeds[1])
-            logging.info("TCP Bandwidth from vm->vm: %s", speeds[2])
+            test.log.info("TCP Bandwidth from vm->host: %s", speeds[0])
+            test.log.info("TCP Bandwidth from host->vm: %s", speeds[1])
+            test.log.info("TCP Bandwidth from vm->vm: %s", speeds[2])
 
             # test udp bandwidth limited to 1Gb
             error_context.context("Test iperf bandwidth udp.")
             speeds = self.test_bandwidth("-u -b 1G")
-            logging.info("UDP Bandwidth from vm->host: %s", speeds[0])
-            logging.info("UDP Bandwidth from host->vm: %s", speeds[1])
-            logging.info("UDP Bandwidth from vm->vm: %s", speeds[2])
+            test.log.info("UDP Bandwidth from vm->host: %s", speeds[0])
+            test.log.info("UDP Bandwidth from host->vm: %s", speeds[1])
+            test.log.info("UDP Bandwidth from vm->vm: %s", speeds[2])
 
         def clean(self, test, params, env):
             self.host.cmd("killall -9 iperf")