attacker_general: fix logging to use %s 37/34237/1
authorRoss Brattain <ross.b.brattain@intel.com>
Tue, 4 Apr 2017 06:43:56 +0000 (23:43 -0700)
committerJing Lu <lvjing5@huawei.com>
Thu, 4 May 2017 12:30:35 +0000 (12:30 +0000)
Change-Id: Ib451d7883eb5df13cfe95477cea43c076ac0452a
Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>
(cherry picked from commit 45f46d5220cbfa89224d55caf94a079fccd2679f)

yardstick/benchmark/scenarios/availability/attacker/attacker_general.py

index 24888af..ab5e998 100644 (file)
@@ -40,19 +40,19 @@ class GeneralAttacker(BaseAttacker):
         if "action_parameter" in self._config:
             actionParameter = self._config['action_parameter']
             str = util.buildshellparams(actionParameter)
-            LOG.debug("inject parameter is: {0}".format(actionParameter))
-            LOG.debug("inject parameter values are: {0}"
-                      .format(list(actionParameter.values())))
-            l = list(item for item in actionParameter.values())
+            LOG.debug("inject parameter is: %s", actionParameter)
+            LOG.debug("inject parameter values are: %s",
+                      list(actionParameter.values()))
+            l = list(actionParameter.values())
             self.action_param = str.format(*l)
 
         if "rollback_parameter" in self._config:
             rollbackParameter = self._config['rollback_parameter']
             str = util.buildshellparams(rollbackParameter)
-            LOG.debug("recover parameter is: {0}".format(rollbackParameter))
-            LOG.debug("recover parameter values are: {0}".
-                      format(list(rollbackParameter.values())))
-            l = list(item for item in rollbackParameter.values())
+            LOG.debug("recover parameter is: %s", rollbackParameter)
+            LOG.debug("recover parameter values are: %s",
+                      list(rollbackParameter.values()))
+            l = list(rollbackParameter.values())
             self.rollback_param = str.format(*l)
 
         self.fault_cfg = BaseAttacker.attacker_cfgs.get(self.attack_key)
@@ -77,10 +77,9 @@ class GeneralAttacker(BaseAttacker):
                     "/bin/bash -s ",
                     stdin=stdin_file)
 
-        LOG.debug("the inject_fault's exit status is: {0}".format(exit_status))
+        LOG.debug("the inject_fault's exit status is: %s", exit_status)
         if exit_status == 0:
-            LOG.debug("success,the inject_fault's output is: {0}"
-                      .format(stdout))
+            LOG.debug("success,the inject_fault's output is: %s", stdout)
         else:
             LOG.error(
                 "the inject_fault's error, stdout:%s, stderr:%s",