vnfdgen: replace six.iteritems() with .items() 37/27637/2
authorRoss Brattain <ross.b.brattain@intel.com>
Thu, 26 Jan 2017 19:26:22 +0000 (11:26 -0800)
committerRoss Brattain <ross.b.brattain@intel.com>
Wed, 8 Feb 2017 06:50:42 +0000 (06:50 +0000)
Openstack python3 style guide prefers to not use six.iteritems() but
instead use data.items() for all cases.

We should follow the Openstack guides since OPNFV is aligned with
Openstack

https://wiki.openstack.org/wiki/Python3#Port_Python_2_code_to_Python_3

Common patterns:
* Replace "for key in dict.iterkeys()" with "for key in dict"
* Replace dict.iteritems() with dict.items()
* Replace dict.itervalues() with dict.values()

Change-Id: I99b0505199a568771341658caf8b71c1896db9cd
Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>
yardstick/network_services/vnf_generic/vnfdgen.py

index 9a02050..64554cd 100644 (file)
@@ -16,7 +16,6 @@
 from __future__ import absolute_import
 import collections
 import yaml
-import six
 
 from yardstick.common.task_template import TaskTemplate
 
@@ -46,13 +45,14 @@ def dict_key_flatten(data):
     """
     next_data = {}
 
+    # check for non-string iterables
     if not any((isinstance(v, collections.Iterable) and not isinstance(v, str))
                for v in data.values()):
         return data
 
-    for key, val in six.iteritems(data):
+    for key, val in data.items():
         if isinstance(val, collections.Mapping):
-            for n_k, n_v in six.iteritems(val):
+            for n_k, n_v in val.items():
                 next_data["%s.%s" % (key, n_k)] = n_v
         elif isinstance(val, collections.Iterable) and not isinstance(val,
                                                                       str):