diff --git a/nova/virt/xenapi/vm_utils.py b/nova/virt/xenapi/vm_utils.py
index 9bad2cdc7088..0c930ec7052f 100644
--- a/nova/virt/xenapi/vm_utils.py
+++ b/nova/virt/xenapi/vm_utils.py
@@ -23,7 +23,6 @@ import contextlib
 import math
 import os
 import time
-import urllib
 from xml.dom import minidom
 from xml.parsers import expat
 
@@ -42,6 +41,7 @@ from oslo_utils import versionutils
 import six
 from six.moves import range
 import six.moves.urllib.parse as urlparse
+import six.moves.urllib.request as urlrequest
 
 from nova.api.metadata import base as instance_metadata
 from nova.compute import power_state
@@ -1950,7 +1950,7 @@ def _get_rrd_server():
 def _get_rrd(server, vm_uuid):
     """Return the VM RRD XML as a string."""
     try:
-        xml = urllib.urlopen("%s://%s:%s@%s/vm_rrd?uuid=%s" % (
+        xml = urlrequest.urlopen("%s://%s:%s@%s/vm_rrd?uuid=%s" % (
             server[0],
             CONF.xenserver.connection_username,
             CONF.xenserver.connection_password,
diff --git a/tools/xenserver/vm_vdi_cleaner.py b/tools/xenserver/vm_vdi_cleaner.py
index 6d868b20252b..27c12663bec9 100755
--- a/tools/xenserver/vm_vdi_cleaner.py
+++ b/tools/xenserver/vm_vdi_cleaner.py
@@ -105,7 +105,7 @@ def _get_applicable_vm_recs(xenapi):
     for vm_ref in call_xenapi(xenapi, 'VM.get_all'):
         try:
             vm_rec = call_xenapi(xenapi, 'VM.get_record', vm_ref)
-        except XenAPI.Failure, e:
+        except XenAPI.Failure as e:
             if e.details[0] != 'HANDLE_INVALID':
                 raise
             continue
@@ -161,7 +161,7 @@ def _find_vdis_connected_to_vm(xenapi, connected_vdi_uuids):
                 try:
                     cur_vdi_rec = call_xenapi(xenapi, 'VDI.get_record',
                             cur_vdi_ref)
-                except XenAPI.Failure, e:
+                except XenAPI.Failure as e:
                     if e.details[0] != 'HANDLE_INVALID':
                         raise
                     break
@@ -176,7 +176,7 @@ def _find_vdis_connected_to_vm(xenapi, connected_vdi_uuids):
         for vbd_ref in vbd_refs:
             try:
                 vbd_rec = call_xenapi(xenapi, 'VBD.get_record', vbd_ref)
-            except XenAPI.Failure, e:
+            except XenAPI.Failure as e:
                 if e.details[0] != 'HANDLE_INVALID':
                     raise
                 continue
@@ -191,7 +191,7 @@ def _find_vdis_connected_to_vm(xenapi, connected_vdi_uuids):
 
             try:
                 vdi_rec = call_xenapi(xenapi, 'VDI.get_record', vbd_vdi_ref)
-            except XenAPI.Failure, e:
+            except XenAPI.Failure as e:
                 if e.details[0] != 'HANDLE_INVALID':
                     raise
                 continue
@@ -210,7 +210,7 @@ def _find_all_vdis_and_system_vdis(xenapi, all_vdi_uuids, connected_vdi_uuids):
     for vdi_ref in call_xenapi(xenapi, 'VDI.get_all'):
         try:
             vdi_rec = call_xenapi(xenapi, 'VDI.get_record', vdi_ref)
-        except XenAPI.Failure, e:
+        except XenAPI.Failure as e:
             if e.details[0] != 'HANDLE_INVALID':
                 raise
             continue
@@ -254,7 +254,7 @@ def clean_orphaned_vdis(xenapi, vdi_uuids):
         vdi_ref = call_xenapi(xenapi, 'VDI.get_by_uuid', vdi_uuid)
         try:
             call_xenapi(xenapi, 'VDI.destroy', vdi_ref)
-        except XenAPI.Failure, exc:
+        except XenAPI.Failure as exc:
             sys.stderr.write("Skipping %s: %s" % (vdi_uuid, exc))