Replace data structures' attribute with six module
In Python 3, some data structures' attribute is different in Python 2. See http://pythonhosted.org/six/#object-model-compatibility This is change mapping: six Python 2 Python 3 six.next(it) it.next() next(it) six.iterkeys(dict) dict.iterkeys() dict.keys() six.itervalues(dict) dict.itervalues() dict.values() Implements: blueprint make-python3-compatible Change-Id: Ida48f39ff230860feee7305b93b134c625a21663
This commit is contained in:
parent
24a3c82d06
commit
16fb43b8b5
|
@ -89,7 +89,7 @@ def load_plugin_from_args(args):
|
|||
plugin.sufficient_options()
|
||||
return plugin
|
||||
|
||||
for plugin_auth_system in sorted(_discovered_plugins.iterkeys()):
|
||||
for plugin_auth_system in sorted(six.iterkeys(_discovered_plugins)):
|
||||
plugin_class = _discovered_plugins[plugin_auth_system]
|
||||
plugin = plugin_class()
|
||||
plugin.parse_opts(args)
|
||||
|
|
|
@ -264,7 +264,7 @@ def _safe_log(log_func, msg, msg_data):
|
|||
|
||||
def _fix_passwords(d):
|
||||
"""Sanitizes the password fields in the dictionary."""
|
||||
for k in d.iterkeys():
|
||||
for k in six.iterkeys(d):
|
||||
if k.lower().find('password') != -1:
|
||||
d[k] = '<SANITIZED>'
|
||||
elif k.lower() in SANITIZE:
|
||||
|
|
|
@ -26,6 +26,7 @@ import kombu.connection
|
|||
import kombu.entity
|
||||
import kombu.messaging
|
||||
from oslo.config import cfg
|
||||
import six
|
||||
|
||||
from openstack.common import excutils
|
||||
from openstack.common.gettextutils import _ # noqa
|
||||
|
@ -623,7 +624,7 @@ class Connection(object):
|
|||
|
||||
def _declare_consumer():
|
||||
consumer = consumer_cls(self.conf, self.channel, topic, callback,
|
||||
self.consumer_num.next())
|
||||
six.next(self.consumer_num))
|
||||
self.consumers.append(consumer)
|
||||
return consumer
|
||||
|
||||
|
@ -730,7 +731,7 @@ class Connection(object):
|
|||
it = self.iterconsume(limit=limit)
|
||||
while True:
|
||||
try:
|
||||
it.next()
|
||||
six.next(it)
|
||||
except StopIteration:
|
||||
return
|
||||
|
||||
|
|
|
@ -21,6 +21,7 @@ import uuid
|
|||
import eventlet
|
||||
import greenlet
|
||||
from oslo.config import cfg
|
||||
import six
|
||||
|
||||
from openstack.common import excutils
|
||||
from openstack.common.gettextutils import _ # noqa
|
||||
|
@ -537,7 +538,7 @@ class Connection(object):
|
|||
consumers = self.consumers
|
||||
self.consumers = {}
|
||||
|
||||
for consumer in consumers.itervalues():
|
||||
for consumer in six.itervalues(consumers):
|
||||
consumer.reconnect(self.session)
|
||||
self._register_consumer(consumer)
|
||||
|
||||
|
@ -695,7 +696,7 @@ class Connection(object):
|
|||
it = self.iterconsume(limit=limit)
|
||||
while True:
|
||||
try:
|
||||
it.next()
|
||||
six.next(it)
|
||||
except StopIteration:
|
||||
return
|
||||
|
||||
|
|
|
@ -23,6 +23,7 @@ import uuid
|
|||
import eventlet
|
||||
import greenlet
|
||||
from oslo.config import cfg
|
||||
import six
|
||||
from six import moves
|
||||
|
||||
from openstack.common import excutils
|
||||
|
@ -522,8 +523,8 @@ def unflatten_envelope(packenv):
|
|||
h = {}
|
||||
try:
|
||||
while True:
|
||||
k = i.next()
|
||||
h[k] = i.next()
|
||||
k = six.next(i)
|
||||
h[k] = six.next(i)
|
||||
except StopIteration:
|
||||
return h
|
||||
|
||||
|
|
|
@ -46,6 +46,7 @@ import subunit
|
|||
import sys
|
||||
import unittest
|
||||
|
||||
import six
|
||||
import testtools
|
||||
|
||||
|
||||
|
@ -273,7 +274,7 @@ class OpenStackTestResult(testtools.TestResult):
|
|||
self.stopTestRun()
|
||||
|
||||
def stopTestRun(self):
|
||||
for cls in list(self.results.iterkeys()):
|
||||
for cls in list(six.iterkeys(self.results)):
|
||||
self.writeTestCase(cls)
|
||||
self.stream.writeln()
|
||||
self.writeSlowTests()
|
||||
|
|
Loading…
Reference in New Issue