Migrating scripts to python3

Author: Your Name

packaging/centos8/cloud.spec

@@ -49,14 +49,17 @@ BuildRequires: /usr/bin/mkisofs
 BuildRequires: maven => 3.0.0
 #BuildRequires: python-setuptools
 
+Autoreq: no
+AutoReqProv: no
+
 %description
 CloudStack is a highly-scalable elastic, open source,
 intelligent IaaS cloud implementation.
 
 %package management
 Summary:   CloudStack management server UI
 Requires: java-11-openjdk
-Requires: python
+Requires: python2
 Requires: python3
 Requires: bash
 Requires: gawk
@@ -74,23 +77,23 @@ Requires: /sbin/service
 Requires: /sbin/chkconfig
 Requires: /usr/bin/ssh-keygen
 Requires: genisoimage
-Requires: mysql-connector-python
+#Requires: mysql-connector-python
 Requires: ipmitool
 Requires: %{name}-common = %{_ver}
 Requires: iptables-services
 Requires: qemu-img
-Requires: python-dns
+Requires: python3-dns
 Group:     System Environment/Libraries
 %description management
 The CloudStack management server is the central point of coordination,
 management, and intelligence in CloudStack.
 
 %package common
 Summary: Apache CloudStack common files and scripts
-Requires: python
+Requires: python2
 Requires: python3
-Requires: python-argparse
-Requires: python-netaddr
+#Requires: python-argparse
+Requires: python3-netaddr
 Group:   System Environment/Libraries
 %description common
 The Apache CloudStack files shared between agent and management server
@@ -110,7 +113,7 @@ Requires: net-tools
 Requires: iproute
 Requires: ipset
 Requires: perl
-Requires: libvirt-python
+Requires: python3-libvirt
 Requires: qemu-img
 Requires: qemu-kvm
 Provides: cloud-agent
@@ -146,9 +149,11 @@ Apache CloudStack command line interface
 
 %package marvin
 Summary: Apache CloudStack Marvin library
-Requires: python-pip
+Requires: python3-pip
+Requires: python2-pip
 Requires: gcc
-Requires: python-devel
+Requires: python3-devel
+Requires: python2-devel
 Requires: libffi-devel
 Requires: openssl-devel
 Group: System Environment/Libraries
@@ -599,3 +604,4 @@ pip install --upgrade /usr/share/cloudstack-marvin/Marvin-*.tar.gz
 
 * Fri Oct 5 2012 Hugo Trippaers <hugo@apache.org> 4.1.0
 - new style spec file
+

scripts/network/exdhcp/dhcpd_edithosts.py

@@ -1,4 +1,4 @@
-#!/usr/bin/python
+#!/usr/bin/python3
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information
@@ -36,27 +36,27 @@ def lock():
 			sleep(1)
 			count = count + 1
 			if count > sleep_max:
-				print "Can not get file lock at %s, time expired" % file_lock
+				print("Can not get file lock at %s, time expired" % file_lock)
 				return False
 
 	try:
 		f = open(file_lock, "w")
 		f.close()
 		return True
-	except IOError,e:
-		print "Cannot create file lock at /etc/dhcpd.conf_locked,", e
+	except IOError as e:
+		print("Cannot create file lock at /etc/dhcpd.conf_locked,", e)
 		return False
 
 
 def unlock():
 	if exists(file_lock) == False:
-		print "Cannot find %s when unlocking, race condition happens" % file_lock
+		print("Cannot find %s when unlocking, race condition happens" % file_lock)
 	else:
 		try:
 			remove(file_lock)
 			return True
-		except IOError, e:
-			print "Cannot remove file lock at %s" % file_lock
+		except IOError as e:
+			print("Cannot remove file lock at %s" % file_lock)
 			return False
 
 def insert_host_entry(mac, ip, hostname, dns, gateway, next_server):
@@ -66,14 +66,14 @@ def insert_host_entry(mac, ip, hostname, dns, gateway, next_server):
 	cmd = 'sed -i /"fixed-address %s"/d %s' % (ip, conf_path)
 	ret = os.system(cmd)
 	if ret != 0:
-		print "Command %s failed" % cmd
+		print("Command %s failed" % cmd)
 		unlock()
 		return 1
 
 	cmd = 'sed -i /"hardware ethernet %s"/d %s' % (mac, conf_path)
 	ret = os.system(cmd)
 	if ret != 0:
-		print "Command %s failed" % cmd
+		print("Command %s failed" % cmd)
 		unlock()
 		return 1
 
@@ -84,14 +84,14 @@ def insert_host_entry(mac, ip, hostname, dns, gateway, next_server):
 	cmd = '''echo '%s' >> %s''' % (entry, conf_path)
 	ret = os.system(cmd)
 	if ret != 0:
-		print "Command %s failed" % cmd
+		print("Command %s failed" % cmd)
 		unlock()
 		return 1
 
 	cmd = 'service dhcpd restart'
 	ret = os.system(cmd)
 	if ret != 0:
-		print "Command %s failed" % cmd
+		print("Command %s failed" % cmd)
 		unlock()
 		return 1
 
@@ -102,7 +102,7 @@ def insert_host_entry(mac, ip, hostname, dns, gateway, next_server):
 
 if __name__ == "__main__":
 	if len(sys.argv) < 7:
-		print usage
+		print(usage)
 		sys.exit(1)
 
 	mac = sys.argv[1]
@@ -115,7 +115,7 @@ def insert_host_entry(mac, ip, hostname, dns, gateway, next_server):
 	if exists(conf_path) == False:
 		conf_path = "/etc/dhcp/dhcpd.conf"
 	if exists(conf_path) == False:
-		print "Cannot find dhcpd.conf"
+		print("Cannot find dhcpd.conf")
 		sys.exit(1)
 
 	ret = insert_host_entry(mac, ip, hostname, dns, gateway, next_server)

scripts/network/ping/baremetal_user_data.py

@@ -1,3 +1,4 @@
+#!/usr/bin/python3
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information

scripts/network/ping/prepare_kickstart_bootfile.py

@@ -1,4 +1,4 @@
-#!/usr/bin/python
+#!/usr/bin/python3
 # Copyright 2012 Citrix Systems, Inc. Licensed under the
 # Apache License, Version 2.0 (the "License"); you may not use this
 # file except in compliance with the License.  Citrix Systems, Inc.
@@ -62,14 +62,14 @@ def prepare():
         f.write(stuff)
         f.close()
         return 0
-    except Exception, e:
-        print e
+    except Exception as e:
+        print(e)
         return 1
 
 
 if __name__ == "__main__":
     if len(sys.argv) < 7:
-        print "Usage: prepare_kickstart_bootfile.py tftp_dir mac kernel initrd ks_file ks_device"
+        print("Usage: prepare_kickstart_bootfile.py tftp_dir mac kernel initrd ks_file ks_device")
         exit(1)
 
     (tftp_dir, mac, kernel, initrd, ks_file, ks_device) = sys.argv[1:]

scripts/network/ping/prepare_kickstart_kernel_initrd.py

@@ -1,4 +1,4 @@
-#!/usr/bin/python
+#!/usr/bin/python3
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information
@@ -26,7 +26,7 @@
 copy_to = None
 
 def cmd(cmdstr, err=True):
-    print cmdstr
+    print(cmdstr)
     if os.system(cmdstr) != 0 and err:
         raise Exception("Failed to run shell command: %s" % cmdstr)
 
@@ -36,7 +36,7 @@ def prepare():
         k = os.path.join(copy_to, "vmlinuz")
         i = os.path.join(copy_to, "initrd.img")
         if os.path.exists(k) and os.path.exists(i):
-            print "Having template(%s) prepared already, skip copying" % copy_to
+            print("Having template(%s) prepared already, skip copying" % copy_to)
             return 0
         else:
             if not os.path.exists(copy_to):
@@ -61,13 +61,13 @@ def copy_from_nfs(src, dst):
 
         copy_from_nfs(kernel, copy_to)
         copy_from_nfs(initrd, copy_to)
-    except Exception, e:
-        print e
+    except Exception as e:
+        print(e)
         return 1
 
 if __name__ == "__main__":
     if len(sys.argv) < 4:
-        print "Usage: prepare_kickstart_kerneal_initrd.py path_to_kernel path_to_initrd path_kernel_initrd_copy_to"
+        print("Usage: prepare_kickstart_kerneal_initrd.py path_to_kernel path_to_initrd path_kernel_initrd_copy_to")
 	sys.exit(1)
 
     (kernel, initrd, copy_to) = sys.argv[1:]

scripts/network/ping/prepare_tftp_bootfile.py

@@ -1,4 +1,4 @@
-#!/usr/bin/python
+#!/usr/bin/python3
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information
@@ -72,14 +72,14 @@ def prepare(is_restore):
         f.write(stuff)
         f.close()
         return 0
-    except Exception, e:
-        print e
+    except Exception as e:
+        print(e)
         return 1
 
 
 if __name__ == "__main__":
     if len(sys.argv) < 12:
-        print "Usage: prepare_tftp_bootfile.py tftp_dir mac cifs_server share directory image_to_restor cifs_username cifs_password ip netmask gateway"
+        print("Usage: prepare_tftp_bootfile.py tftp_dir mac cifs_server share directory image_to_restor cifs_username cifs_password ip netmask gateway")
         exit(1)
 
     (cmd, tftp_dir, mac, cifs_server, share, directory, template_dir, cifs_username, cifs_password, ip, netmask, gateway) = sys.argv[1:]
@@ -89,7 +89,7 @@ def prepare(is_restore):
     elif cmd == "backup":
         ret = prepare(False)
     else:
-        print "Unknown cmd: %s"%cmd
+        print("Unknown cmd: %s"%cmd)
         ret = 1
 
     exit(ret)

scripts/storage/secondary/cloud-install-sys-tmplt.py

@@ -1,4 +1,4 @@
-#!/usr/bin/python
+#!/usr/bin/python3
 # -*- coding: utf-8 -*-
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements.  See the NOTICE file
@@ -19,7 +19,7 @@
 
 import argparse
 import sys
-import urllib
+import urllib.request, urllib.parse, urllib.error
 import uuid
 import subprocess
 import os
@@ -90,7 +90,7 @@ def populateOptions(self):
       self.databaseuserpassword = ""
     if self.args.templatesuffix:
       self.templatesuffix = self.args.templatesuffix
-    print 'Password for DB: %s'%self.databaseuserpassword
+    print('Password for DB: %s'%self.databaseuserpassword)
 
   def errorAndExit(self, msg):
     err = '''\n\nWe apologize for below error:
@@ -117,11 +117,11 @@ def runCmd(self, cmds):
 
   def runMysql(self, query):
     try:
-      print 'Running Query: %s' % query
+      print('Running Query: %s' % query)
       mysqlCmds = ['mysql', '--user=%s'%self.databaseusername, '--host=%s'%self.databasehostname, '--password=%s'%self.databaseuserpassword, '--skip-column-names', '-U', 'cloud', '-e "%s"'%query]
       templateId = self.runCmd(mysqlCmds)
-      print 'TemplateId is : %s' % templateId
-    except Exception, e:
+      print('TemplateId is : %s' % templateId)
+    except Exception as e:
       err = '''Encountering an error when executing mysql script\n%s''' % str(e)
       self.errorAndExit(err)
     return templateId
@@ -137,9 +137,9 @@ def fetchTemplateDetails(self):
 
   def downloadTemplate(self):
     self.systemvmtemplatepath = self.templateName + "." + self.fileextension
-    print 'Downloading template from %s To %s' % (self.systemvmtemplateurl, self.systemvmtemplatepath)
+    print('Downloading template from %s To %s' % (self.systemvmtemplateurl, self.systemvmtemplatepath))
     try:
-      templateFileDownloadUrl = urllib.urlretrieve(self.systemvmtemplateurl, self.systemvmtemplatepath, reporthook=self.report)
+      templateFileDownloadUrl = urllib.request.urlretrieve(self.systemvmtemplateurl, self.systemvmtemplatepath, reporthook=self.report)
     except Exception:
       self.errorAndExit("Failed to download template file from %s" % self.systemvmtemplateurl)
 
@@ -150,23 +150,23 @@ def report(tmp, blocknr, blocksize, size):
   def installTemplate(self):
     destDir = self.mountpoint + os.sep + "template" + os.sep + "tmpl" + os.sep + "1" + os.sep + str(self.template)
     self.destDir = destDir
-    print 'The desination Directory is : %s' % destDir
+    print('The desination Directory is : %s' % destDir)
     try:
       if self.forcecleanup:
         if os.path.exists(destDir):
           shutil.rmtree(destDir)
       if not os.path.exists(destDir):
         os.makedirs(destDir)
-    except Exception, e:
+    except Exception as e:
       self.errorAndExit('Failed to create directories on the mounted path.. %s' % str (e))
-    print 'Installing Template to : %s' % destDir
+    print('Installing Template to : %s' % destDir)
     tmpFile = self.templateName + "." + "tmp"
     self.uncompressFile(tmpFile)
-    print 'Moving the decompressed file to destination directory %s... which could take a long time, please wait' % destDir
+    print('Moving the decompressed file to destination directory %s... which could take a long time, please wait' % destDir)
     shutil.move(tmpFile, destDir + os.sep + self.templateName)
 
   def uncompressFile(self, fileName):
-    print 'Uncompressing the file %s... which could take a long time, please wait' % self.systemvmtemplatepath
+    print('Uncompressing the file %s... which could take a long time, please wait' % self.systemvmtemplatepath)
     if self.fileextension == 'gz':
       compressedFile = gzip.GzipFile(self.systemvmtemplatepath, 'rb')
       decompressedData = compressedFile.read()
@@ -181,7 +181,7 @@ def uncompressFile(self, fileName):
       decompressedFile = file(fileName, 'wb')
       decompressedFile.write(decompressedData)
       decompressedFile.close()
-      print ''
+      print('')
     elif self.fileextension == 'zip':
       zippedFile = zipfile.ZipFile(self.systemvmtemplatepath, 'r')
       zippedFiles = zippedFile.namelist()
@@ -191,7 +191,7 @@ def uncompressFile(self, fileName):
       decompressedFile.write(decompressedData)
       decompressedFile.close()
       zippedFile.close()
-      print ''
+      print('')
     else:
       self.errorAndExit('Not supported file type %s to decompress' % self.fileextension)
     self.fileSize = os.path.getsize(fileName)
@@ -226,11 +226,11 @@ def run(self):
       self.installTemplate()
       self.writeProperties()
     finally:
-      print ''
-    print ''
-    print "CloudStack has successfully installed system template"
-    print ''
+      print('')
+    print('')
+    print("CloudStack has successfully installed system template")
+    print('')
 
 if __name__ == "__main__":
    o = InstallSysTemplate()
-   o.run()
+   o.run()