Dan Kenigsberg has uploaded a new change for review.
Change subject: py3: make gluster_cli_tests pass
......................................................................
py3: make gluster_cli_tests pass
Change-Id: I7e212e247f057f2988debaf3b2de923851b90b24
Signed-off-by: Dan Kenigsberg <danken(a)redhat.com>
---
M lib/vdsm/kaxmlrpclib.py
M lib/vdsm/network/netlink/addr.py
M tests/gluster_cli_tests.py
M vdsm/gluster/storagedev.py
4 files changed, 11 insertions(+), 9 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/08/65008/1
diff --git a/lib/vdsm/kaxmlrpclib.py b/lib/vdsm/kaxmlrpclib.py
index e24316c..66ec9e0 100644
--- a/lib/vdsm/kaxmlrpclib.py
+++ b/lib/vdsm/kaxmlrpclib.py
@@ -33,8 +33,8 @@
from __future__ import absolute_import
from __future__ import print_function
-import xmlrpclib
-import httplib
+import six.moves.xmlrpc_client
+import six.moves.http_client
import socket
# It would have been nicer to make these server-specific and not module-wide
@@ -48,19 +48,19 @@
def Server(url, *args, **kwargs):
kwargs['transport'] = TcpkeepTransport()
- server = xmlrpclib.Server(url, *args, **kwargs)
+ server = six.moves.xmlrpc_client.Server(url, *args, **kwargs)
return server
ServerProxy = Server
-class TcpkeepTransport(xmlrpclib.Transport):
+class TcpkeepTransport(six.moves.xmlrpc_client.Transport):
def make_connection(self, host):
return TcpkeepHTTPConnection(host)
-class TcpkeepHTTPConnection(httplib.HTTPConnection):
+class TcpkeepHTTPConnection(six.moves.http_client.HTTPConnection):
def connect(self):
"""Connect to the host and port specified in __init__.
@@ -110,7 +110,7 @@
def SslServer(url, ctx, *args, **kwargs):
kwargs['transport'] = TcpkeepSafeTransport(ctx)
- server = xmlrpclib.Server(url, *args, **kwargs)
+ server = six.moves.xmlrpc_client.Server(url, *args, **kwargs)
return server
SslServerProxy = SslServer
diff --git a/lib/vdsm/network/netlink/addr.py b/lib/vdsm/network/netlink/addr.py
index 0037454..cb17bf3 100644
--- a/lib/vdsm/network/netlink/addr.py
+++ b/lib/vdsm/network/netlink/addr.py
@@ -79,7 +79,7 @@
"""Returns the textual representation of the address
flags"""
flags = (c_char * (CHARBUFFSIZE * 2))()
return frozenset(_rtnl_addr_flags2str(_rtnl_addr_get_flags(addr), flags,
- sizeof(flags)).split(','))
+ sizeof(flags)).split(b','))
# C function prototypes
diff --git a/tests/gluster_cli_tests.py b/tests/gluster_cli_tests.py
index 4ab9daf..0493c27 100644
--- a/tests/gluster_cli_tests.py
+++ b/tests/gluster_cli_tests.py
@@ -18,6 +18,8 @@
# Refer to the README and COPYING files for full details of the license
#
+import six
+
from testlib import VdsmTestCase as TestCaseBase
from gluster import cli as gcli
import xml.etree.cElementTree as etree
@@ -446,7 +448,7 @@
"""
tree = etree.fromstring(out)
status = gcli._parseVolumeStatusClients(tree)
- self.assertEquals(status.keys(), ['bricks', 'name'])
+ self.assertEquals(set(six.iterkeys(status)), {'bricks', 'name'})
self.assertEquals(status['name'], 'music')
oBricks = [{'brick': '192.168.122.2:/tmp/music-b1',
'hostuuid':
diff --git a/vdsm/gluster/storagedev.py b/vdsm/gluster/storagedev.py
index a7ab985..a0eefaa 100644
--- a/vdsm/gluster/storagedev.py
+++ b/vdsm/gluster/storagedev.py
@@ -35,7 +35,7 @@
from vdsm import utils
from vdsm.gluster import exception as ge
-import fstab
+from . import fstab
from . import gluster_mgmt_api
--
To view, visit
https://gerrit.ovirt.org/65008
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I7e212e247f057f2988debaf3b2de923851b90b24
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com>