Avoid the ValueError exception for a ping timeout.

In ping.py:
If ping a ip address timeout, the return value "stdout"
is null. And the code "rtt=float(stdout)" raises a
ValueError exception. The better handle is to avoid the
exception and log a ping timeout.

Change-Id: Ia2a3ff44c35dda1a700a5109f8e642a636e89bc6
Signed-off-by: wu.zhihui <wu.zhihui1@zte.com.cn>

diff --git a/yardstick/benchmark/scenarios/networking/ping.py b/yardstick/benchmark/scenarios/networking/ping.py
index 34278b9..c62c79e 100644 (file)
--- a/yardstick/benchmark/scenarios/networking/ping.py
+++ b/yardstick/benchmark/scenarios/networking/ping.py
@@ -67,12 +67,15 @@ class Ping(base.Scenario):
         if exit_status != 0:
             raise RuntimeError(stderr)
 
-        result["rtt"] = float(stdout)
+        if stdout:
+            result["rtt"] = float(stdout)
 
-        if "sla" in self.scenario_cfg:
-            sla_max_rtt = int(self.scenario_cfg["sla"]["max_rtt"])
-            assert result["rtt"] <= sla_max_rtt, "rtt %f > sla:max_rtt(%f); " % \
-                (result["rtt"], sla_max_rtt)
+            if "sla" in self.scenario_cfg:
+                sla_max_rtt = int(self.scenario_cfg["sla"]["max_rtt"])
+                assert result["rtt"] <= sla_max_rtt, "rtt %f > sla:max_rtt(%f); " % \
+                    (result["rtt"], sla_max_rtt)
+        else:
+            LOG.error("ping '%s' '%s' timeout", options, destination)
 
 
 def _test():