From: Ross Brattain <ross.b.brattain@intel.com>
Date: Sun, 8 Jan 2017 09:02:44 +0000 (-0800)
Subject: heat: minor fixes, don't use len > 0 and use next
X-Git-Tag: danube.1.0~82^2
X-Git-Url: https://gerrit.opnfv.org/gerrit/gitweb?a=commitdiff_plain;h=b82105f8bf793e78f8c144c1d833989cb14d944b;p=yardstick.git

heat: minor fixes, don't use len > 0 and use next
instead of [0]

Change-Id: I1785b7ca3543a8d44eb68de61f4f29063a9c61c7
Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>
---

diff --git a/yardstick/benchmark/contexts/heat.py b/yardstick/benchmark/contexts/heat.py
index 0b2fbdcd6..a9d9e5fc0 100644
--- a/yardstick/benchmark/contexts/heat.py
+++ b/yardstick/benchmark/contexts/heat.py
@@ -220,9 +220,9 @@ class HeatContext(Context):
 
         # copy some vital stack output into server objects
         for server in self.servers:
-            if len(server.ports) > 0:
+            if server.ports:
                 # TODO(hafe) can only handle one internal network for now
-                port = list(server.ports.values())[0]
+                port = next(iter(server.ports.values()))
                 server.private_ip = self.stack.outputs[port["stack_name"]]
 
             if server.floating_ip: