From 86d16d2a98af540ba458721edc2ac8a530c01635 Mon Sep 17 00:00:00 2001 From: kshithijiyer Date: Wed, 8 Jan 2020 15:53:31 +0530 Subject: [py2to3] Change str to list in isinstance() Use 'list' object type in comparisons instead of 'str' Because it is differently treated in py2 and py3. Example: In py2 isinstance(u'foo', str) is False In py3 isinstance(u'foo', str) is True Change-Id: I7663d42494bf59d74550ff4897379d35cc357db4 Signed-off-by: kshithijiyer --- glustolibs-gluster/glustolibs/gluster/tiering_ops.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'glustolibs-gluster/glustolibs/gluster/tiering_ops.py') diff --git a/glustolibs-gluster/glustolibs/gluster/tiering_ops.py b/glustolibs-gluster/glustolibs/gluster/tiering_ops.py index 7e48a48c7..357b3d471 100644 --- a/glustolibs-gluster/glustolibs/gluster/tiering_ops.py +++ b/glustolibs-gluster/glustolibs/gluster/tiering_ops.py @@ -48,7 +48,7 @@ def add_extra_servers_to_cluster(mnode, extra_servers): add_extra_servers_to_cluster("abc.com", ['peer_node1','peer_node2']) """ - if isinstance(extra_servers, str): + if not isinstance(extra_servers, list): extra_servers = [extra_servers] ret = start_glusterd(servers=extra_servers) @@ -97,7 +97,7 @@ def tier_attach(mnode, volname, num_bricks_to_add, extra_servers, tier_attach("abc.com", testvol, '2', ['extra_server1','extra_server2'], extra_server_info) """ - if isinstance(extra_servers, str): + if not isinstance(extra_servers, list): extra_servers = [extra_servers] replica = int(replica) -- cgit