Remove six use from cloudbaseinit

Change-Id: Ia4b4add954ee5192dbf437b415d8e698c0a271b8

Author: kulikjak

cloudbaseinit/conf/base.py

@@ -14,11 +14,8 @@
 
 import abc
 
-import six
 
-
-@six.add_metaclass(abc.ABCMeta)
-class Options(object):
+class Options(object, metaclass=abc.ABCMeta):
 
     """Contact class for all the collections of config options."""
 

cloudbaseinit/metadata/services/azureservice.py

@@ -13,13 +13,13 @@
 #    under the License.
 
 import contextlib
+import io
 import os
 import socket
 import time
 from xml.etree import ElementTree
 
 from oslo_log import log as oslo_logging
-import six
 import untangle
 
 from cloudbaseinit import conf as cloudbaseinit_conf
@@ -115,13 +115,13 @@ def _wire_server_request(self, path, data_xml=None, headers=None,
                 path, data_xml, headers=all_headers))
 
         if parse_xml:
-            return untangle.parse(six.StringIO(encoding.get_as_string(data)))
+            return untangle.parse(io.StringIO(encoding.get_as_string(data)))
         else:
             return data
 
     @staticmethod
     def _encode_xml(xml_root):
-        bio = six.BytesIO()
+        bio = io.BytesIO()
         ElementTree.ElementTree(xml_root).write(
             bio, encoding='utf-8', xml_declaration=True)
         return bio.getvalue()

cloudbaseinit/metadata/services/base.py

@@ -21,7 +21,6 @@
 
 from oslo_log import log as oslo_logging
 import requests
-import six
 
 from cloudbaseinit import conf as cloudbaseinit_conf
 from cloudbaseinit import exception
@@ -35,8 +34,7 @@ class NotExistingMetadataException(Exception):
     pass
 
 
-@six.add_metaclass(abc.ABCMeta)
-class BaseMetadataService(object):
+class BaseMetadataService(object, metaclass=abc.ABCMeta):
     _GZIP_MAGIC_NUMBER = b'\x1f\x8b'
 
     def __init__(self):

cloudbaseinit/metadata/services/cloudstack.py

@@ -13,11 +13,11 @@
 #    under the License.
 
 import contextlib
+import http.client
 import posixpath
+import urllib
 
 from oslo_log import log as oslo_logging
-from six.moves import http_client
-from six.moves import urllib
 
 from cloudbaseinit import conf as cloudbaseinit_conf
 from cloudbaseinit.metadata.services import base
@@ -131,12 +131,12 @@ def get_public_keys(self):
     def _password_client(self, body=None, headers=None, decode=True):
         """Client for the Password Server."""
         port = CONF.cloudstack.password_server_port
-        with contextlib.closing(http_client.HTTPConnection(
+        with contextlib.closing(http.client.HTTPConnection(
                 self._metadata_host, port, timeout=TIMEOUT)) as connection:
             try:
                 connection.request("GET", "/", body=body, headers=headers)
                 response = connection.getresponse()
-            except http_client.HTTPException as exc:
+            except http.client.HTTPException as exc:
                 LOG.error("Request failed: %s", exc)
                 raise
 
@@ -145,7 +145,7 @@ def _password_client(self, body=None, headers=None, decode=True):
                 content = encoding.get_as_string(content)
 
             if response.status != 200:
-                raise http_client.HTTPException(
+                raise http.client.HTTPException(
                     "%(status)s %(reason)s - %(message)r",
                     {"status": response.status, "reason": response.reason,
                      "message": content})

cloudbaseinit/metadata/services/httpservice.py

@@ -12,8 +12,9 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
+from urllib import error
+
 from oslo_log import log as oslo_logging
-from six.moves.urllib import error
 
 from cloudbaseinit import conf as cloudbaseinit_conf
 from cloudbaseinit.metadata.services import base

cloudbaseinit/metadata/services/opennebulaservice.py

@@ -21,7 +21,6 @@
 import struct
 
 from oslo_log import log as oslo_logging
-import six
 
 from cloudbaseinit.metadata.services import base
 from cloudbaseinit.models import network as network_model
@@ -108,8 +107,7 @@ def _calculate_netmask(address, gateway):
         address_chunks = address.split(".")
         gateway_chunks = gateway.split(".")
         netmask_chunks = []
-        for achunk, gchunk in six.moves.zip(
-                address_chunks, gateway_chunks):
+        for achunk, gchunk in zip(address_chunks, gateway_chunks):
             if achunk == gchunk:
                 nchunk = "255"
             else:

cloudbaseinit/metadata/services/osconfigdrive/base.py

@@ -15,11 +15,8 @@
 import abc
 import tempfile
 
-import six
 
-
-@six.add_metaclass(abc.ABCMeta)
-class BaseConfigDriveManager(object):
+class BaseConfigDriveManager(object, metaclass=abc.ABCMeta):
 
     def __init__(self):
         self.target_path = tempfile.mkdtemp()

cloudbaseinit/metadata/services/packet.py

@@ -15,12 +15,12 @@
 
 import json
 import requests
+from urllib import error
 
 from cloudbaseinit import conf as cloudbaseinit_conf
 from cloudbaseinit import exception
 from cloudbaseinit.metadata.services import base
 from oslo_log import log as oslo_logging
-from six.moves.urllib import error
 
 CONF = cloudbaseinit_conf.CONF
 LOG = oslo_logging.getLogger(__name__)

cloudbaseinit/osutils/windows.py

@@ -20,12 +20,11 @@
 import struct
 import subprocess
 import time
+import winreg
 
 import netaddr
 from oslo_log import log as oslo_logging
 import pywintypes
-import six
-from six.moves import winreg
 from tzlocal import windows_tz
 import win32api
 from win32com import client
@@ -602,7 +601,7 @@ def _get_user_sid_and_domain(self, username):
         sidNameUse = wintypes.DWORD()
 
         ret_val = advapi32.LookupAccountNameW(
-            0, six.text_type(username), sid, ctypes.byref(cbSid), domainName,
+            0, str(username), sid, ctypes.byref(cbSid), domainName,
             ctypes.byref(cchReferencedDomainName), ctypes.byref(sidNameUse))
         if not ret_val:
             raise exception.WindowsCloudbaseInitException(
@@ -613,9 +612,9 @@ def _get_user_sid_and_domain(self, username):
     def add_user_to_local_group(self, username, groupname):
 
         lmi = Win32_LOCALGROUP_MEMBERS_INFO_3()
-        lmi.lgrmi3_domainandname = six.text_type(username)
+        lmi.lgrmi3_domainandname = str(username)
 
-        ret_val = netapi32.NetLocalGroupAddMembers(0, six.text_type(groupname),
+        ret_val = netapi32.NetLocalGroupAddMembers(0, str(groupname),
                                                    3, ctypes.pointer(lmi), 1)
 
         if ret_val == self.NERR_GroupNotFound:
@@ -652,9 +651,9 @@ def create_user_logon_session(self, username, password, domain='.',
                   {"username": username, "domain": domain})
 
         token = wintypes.HANDLE()
-        ret_val = advapi32.LogonUserW(six.text_type(username),
-                                      six.text_type(domain),
-                                      six.text_type(password),
+        ret_val = advapi32.LogonUserW(str(username),
+                                      str(domain),
+                                      str(password),
                                       logon_type,
                                       self.LOGON32_PROVIDER_DEFAULT,
                                       ctypes.byref(token))
@@ -665,7 +664,7 @@ def create_user_logon_session(self, username, password, domain='.',
         if load_profile:
             pi = Win32_PROFILEINFO()
             pi.dwSize = ctypes.sizeof(Win32_PROFILEINFO)
-            pi.lpUserName = six.text_type(username)
+            pi.lpUserName = str(username)
             ret_val = userenv.LoadUserProfileW(token, ctypes.byref(pi))
             if not ret_val:
                 kernel32.CloseHandle(token)
@@ -756,8 +755,7 @@ def sanitize_shell_input(self, value):
 
     def set_host_name(self, new_host_name):
         ret_val = kernel32.SetComputerNameExW(
-            self.ComputerNamePhysicalDnsHostname,
-            six.text_type(new_host_name))
+            self.ComputerNamePhysicalDnsHostname, str(new_host_name))
         if not ret_val:
             raise exception.WindowsCloudbaseInitException(
                 "Cannot set host name: %r")
@@ -1401,7 +1399,7 @@ def check_os_version(self, major, minor, build=0):
     def get_volume_label(self, drive):
         max_label_size = 261
         label = ctypes.create_unicode_buffer(max_label_size)
-        ret_val = kernel32.GetVolumeInformationW(six.text_type(drive), label,
+        ret_val = kernel32.GetVolumeInformationW(str(drive), label,
                                                  max_label_size, 0, 0, 0, 0, 0)
         if ret_val:
             return label.value
@@ -1411,8 +1409,7 @@ def get_volume_path_names_by_mount_point(self, mount_point):
         volume_name = ctypes.create_unicode_buffer(max_volume_name_len)
 
         if not kernel32.GetVolumeNameForVolumeMountPointW(
-                six.text_type(mount_point), volume_name,
-                max_volume_name_len):
+                str(mount_point), volume_name, max_volume_name_len):
             if kernel32.GetLastError() in [self.ERROR_INVALID_NAME,
                                            self.ERROR_PATH_NOT_FOUND]:
                 raise exception.ItemNotFoundException(

cloudbaseinit/plugins/common/createuser.py

@@ -15,7 +15,6 @@
 import abc
 
 from oslo_log import log as oslo_logging
-import six
 
 from cloudbaseinit import conf as cloudbaseinit_conf
 from cloudbaseinit.osutils import factory as osutils_factory
@@ -26,8 +25,7 @@
 LOG = oslo_logging.getLogger(__name__)
 
 
-@six.add_metaclass(abc.ABCMeta)
-class BaseCreateUserPlugin(base.BasePlugin):
+class BaseCreateUserPlugin(base.BasePlugin, metaclass=abc.ABCMeta):
     """This is a base class for creating or modifying an user."""
 
     @abc.abstractmethod