We use foo_bar across the project (except the main program), though
it might be not good to use "foo_bar" as the module name in Python,
but let's unify it first.
---
.../en-US/Understanding_libvirt-test-API.xml | 4 +-
.../en-US/Writing_a_test_case.xml | 8 ++--
libvirt-test-api | 8 ++--
repos/domain/attach_disk.py | 4 +-
repos/domain/attach_interface.py | 4 +-
repos/domain/create.py | 4 +-
repos/domain/define.py | 4 +-
repos/domain/detach_disk.py | 4 +-
repos/domain/detach_interface.py | 4 +-
repos/domain/install_image.py | 4 +-
repos/domain/install_linux_cdrom.py | 6 +-
repos/domain/install_linux_net.py | 6 +-
repos/domain/install_windows_cdrom.py | 6 +-
repos/domain/migrate.py | 2 +-
repos/domain/update_devflag.py | 6 +-
repos/interface/create.py | 2 +-
repos/interface/define.py | 4 +-
repos/interface/destroy.py | 2 +-
repos/interface/undefine.py | 2 +-
repos/network/create.py | 4 +-
repos/network/define.py | 4 +-
repos/npiv/create_virtual_hba.py | 4 +-
.../multiple_thread_block_on_domain_create.py | 4 +-
repos/snapshot/internal_create.py | 4 +-
repos/storage/activate_pool.py | 2 +-
repos/storage/create_dir_pool.py | 4 +-
repos/storage/create_dir_volume.py | 4 +-
repos/storage/create_fs_pool.py | 8 ++--
repos/storage/create_iscsi_pool.py | 4 +-
repos/storage/create_logical_volume.py | 4 +-
repos/storage/create_netfs_pool.py | 8 ++--
repos/storage/create_netfs_volume.py | 4 +-
repos/storage/create_partition_volume.py | 4 +-
repos/storage/define_dir_pool.py | 4 +-
repos/storage/define_disk_pool.py | 4 +-
repos/storage/define_iscsi_pool.py | 4 +-
repos/storage/define_logical_pool.py | 4 +-
repos/storage/define_mpath_pool.py | 4 +-
repos/storage/define_netfs_pool.py | 4 +-
repos/storage/define_scsi_pool.py | 4 +-
repos/storage/delete_partition_volume.py | 2 +-
repos/storage/destroy_pool.py | 2 +-
src/logxmlparser.py | 2 +-
utils/{xmlbuilder.py => xml_builder.py} | 44 ++++++++++----------
utils/{xmlgenerator.py => xml_generator.py} | 2 +-
utils/{XMLParser.py => xml_parser.py} | 8 ++--
46 files changed, 117 insertions(+), 117 deletions(-)
rename utils/{xmlbuilder.py => xml_builder.py} (91%)
rename utils/{xmlgenerator.py => xml_generator.py} (99%)
rename utils/{XMLParser.py => xml_parser.py} (96%)
diff --git a/docs/User_Guide/libvirt-test-API_Guide/en-US/Understanding_libvirt-test-API.xml b/docs/User_Guide/libvirt-test-API_Guide/en-US/Understanding_libvirt-test-API.xml
index 7fe1e97..88c1b76 100644
--- a/docs/User_Guide/libvirt-test-API_Guide/en-US/Understanding_libvirt-test-API.xml
+++ b/docs/User_Guide/libvirt-test-API_Guide/en-US/Understanding_libvirt-test-API.xml
@@ -197,10 +197,10 @@ repos
<para><filename>/utils</filename> is a directory which contains various scripts to assist in creating and verifying test cases.</para>
<!--<itemizedlist>
<listitem>
-<para><filename>xmlgenerator.py</filename>XML generator.</para>
+<para><filename>xml_generator.py</filename>XML generator.</para>
</listitem>
<listitem>
-<para><filename>xmlbuilder.py</filename> builds a virtual machine from an XML file.</para>
+<para><filename>xml_builder.py</filename> builds a virtual machine from an XML file.</para>
</listitem>
<listitem>
<para>Random MAC address generator. Useful when installing a guest machine.</para>
diff --git a/docs/User_Guide/libvirt-test-API_Guide/en-US/Writing_a_test_case.xml b/docs/User_Guide/libvirt-test-API_Guide/en-US/Writing_a_test_case.xml
index 21c4ac6..2a74518 100644
--- a/docs/User_Guide/libvirt-test-API_Guide/en-US/Writing_a_test_case.xml
+++ b/docs/User_Guide/libvirt-test-API_Guide/en-US/Writing_a_test_case.xml
@@ -142,7 +142,7 @@ import exception
from lib import connectAPI
from lib import storageAPI
from utils import env_parser
-from utils import xmlbuilder
+from utils import xml_builder
envfile = 'env.ini'
@@ -154,7 +154,7 @@ def initialize_storage(dict):
dict['sourcepath'] = envparser.get_value('storage', 'sourcepath')
logger.info('prepare create storage pool')
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
poolxml = xmlobj.build_pool(dict)
logger.debug('dump create storage pool xml:\n%s' %poolxml)
@@ -201,7 +201,7 @@ import exception
from lib import connectAPI
from lib import domainAPI
from utils import env_parser
-from utils import xmlbuilder
+from utils import xml_builder
envfile = 'env.ini'
@@ -246,7 +246,7 @@ def install_guest(dict):
logger.info('prepare installation booting cdrom')
prepare_cdrom(url, dict['kickstart'], gname, logger)
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
guestinstxml = xmlobj.build_domain_install(dict)
logger.debug('dump installation guest xml:\n%s' %guestinstxml)
diff --git a/libvirt-test-api b/libvirt-test-api
index 7eaaf21..62e18f6 100755
--- a/libvirt-test-api
+++ b/libvirt-test-api
@@ -28,7 +28,7 @@ from src import generator
from src import env_clear
from src import process
from utils import log
-from src.logxmlparser import LogXMLParser
+from src.logxmlparser import Logxml_parser
from src.activityfilter import Filter
from src.casecfgcheck import CaseCfgCheck
@@ -93,7 +93,7 @@ class Main(object):
testrunid = time.strftime("%Y%m%d%H%M%S")
os.makedirs('log/%s' %testrunid)
- log_xml_parser = LogXMLParser(self.logxml)
+ log_xml_parser = Logxml_parser(self.logxml)
# If the specified log xmlfile exists, then append the testrun
# item of this time to the file, if not, create a new log xmlfile
@@ -256,7 +256,7 @@ class Main(object):
def remove_log(self, testrunid, testid = None):
""" to remove log item in the log xmlfile """
- log_xml_parser = LogXMLParser(self.logxml)
+ log_xml_parser = Logxml_parser(self.logxml)
# remove a test in a testrun
if testrunid and testid:
@@ -289,7 +289,7 @@ class Main(object):
def merge_logxmls(self, logxml_two):
""" to merge two log xml files of log into one"""
- log_xml_parser = LogXMLParser(self.logxml)
+ log_xml_parser = Logxml_parser(self.logxml)
log_xml_parser.merge_xmlfiles(logxml_two)
print "Merge the second log xml file %s to %s successfully " % \
(logxml_two, self.logxml)
diff --git a/repos/domain/attach_disk.py b/repos/domain/attach_disk.py
index fc0e611..4711ad1 100644
--- a/repos/domain/attach_disk.py
+++ b/repos/domain/attach_disk.py
@@ -12,7 +12,7 @@ from libvirt import libvirtError
from src import sharedmod
from utils import utils
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('guestname',
'virt_type',
@@ -68,7 +68,7 @@ def attach_disk(params):
domobj = conn.lookupByName(guestname)
# Generate disk xml
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
diskxml = xmlobj.build_disk(params)
logger.debug("disk xml:\n%s" %diskxml)
diff --git a/repos/domain/attach_interface.py b/repos/domain/attach_interface.py
index 5c9c507..4d605f9 100644
--- a/repos/domain/attach_interface.py
+++ b/repos/domain/attach_interface.py
@@ -10,7 +10,7 @@ from libvirt import libvirtError
from src import sharedmod
from utils import utils
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('guestname', 'ifacetype', 'source',)
optional_params = ('hdmodel',)
@@ -40,7 +40,7 @@ def attach_interface(params):
domobj = conn.lookupByName(guestname)
# Generate interface xml
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
interfacexml = xmlobj.build_interface(params)
logger.debug("interface xml:\n%s" %interfacexml)
diff --git a/repos/domain/create.py b/repos/domain/create.py
index 9f2f5e1..ed96b36 100644
--- a/repos/domain/create.py
+++ b/repos/domain/create.py
@@ -11,7 +11,7 @@ from libvirt import libvirtError
from src import sharedmod
from utils import utils
-from utils import xmlbuilder
+from utils import xml_builder
NONE = 0
START_PAUSED = 1
@@ -43,7 +43,7 @@ def create(params):
conn = sharedmod.libvirtobj['conn']
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
domain = xmlobj.add_domain(params)
xmlobj.add_disk(params, domain)
xmlobj.add_interface(params, domain)
diff --git a/repos/domain/define.py b/repos/domain/define.py
index dae1062..92a98f6 100644
--- a/repos/domain/define.py
+++ b/repos/domain/define.py
@@ -12,7 +12,7 @@ from libvirt import libvirtError
from src import sharedmod
from utils import utils
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('guestname', 'virt_type',)
optional_params = ('uuid',
@@ -70,7 +70,7 @@ def define(params):
logger.info("define domain on %s" % uri)
# Generate damain xml
- xml_obj = xmlbuilder.XmlBuilder()
+ xml_obj = xml_builder.XmlBuilder()
domain = xml_obj.add_domain(params)
xml_obj.add_disk(params, domain)
xml_obj.add_interface(params, domain)
diff --git a/repos/domain/detach_disk.py b/repos/domain/detach_disk.py
index da25c8f..d723474 100644
--- a/repos/domain/detach_disk.py
+++ b/repos/domain/detach_disk.py
@@ -11,7 +11,7 @@ from libvirt import libvirtError
from src import sharedmod
from utils import utils
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('guestname', 'virt_type', 'imagename', 'hdmodel',)
optional_params = ()
@@ -45,7 +45,7 @@ def detach_disk(params):
domobj = conn.lookupByName(guestname)
# Detach disk
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
diskxml = xmlobj.build_disk(params)
logger.debug("disk xml:\n%s" %diskxml)
diff --git a/repos/domain/detach_interface.py b/repos/domain/detach_interface.py
index c444b68..82deffb 100644
--- a/repos/domain/detach_interface.py
+++ b/repos/domain/detach_interface.py
@@ -11,7 +11,7 @@ from libvirt import libvirtError
from src import sharedmod
from utils import utils
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('guestname', 'ifacetype', 'source', 'nicmodel',)
optional_params = ()
@@ -47,7 +47,7 @@ def detach_interface(params):
conn = sharedmod.libvirtobj['conn']
domobj = conn.lookupByName(guestname)
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
ifacexml = xmlobj.build_interface(params)
logger.debug("interface xml:\n%s" % ifacexml)
diff --git a/repos/domain/install_image.py b/repos/domain/install_image.py
index 2ab97a0..be36b03 100644
--- a/repos/domain/install_image.py
+++ b/repos/domain/install_image.py
@@ -13,7 +13,7 @@ from libvirt import libvirtError
from src import sharedmod
from src import env_parser
from utils import utils
-from utils import xmlbuilder
+from utils import xml_builder
HOME_PATH = os.getcwd()
@@ -72,7 +72,7 @@ def install_image(params):
conn = sharedmod.libvirtobj['conn']
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
domain = xmlobj.add_domain(params)
xmlobj.add_disk(params, domain)
diff --git a/repos/domain/install_linux_cdrom.py b/repos/domain/install_linux_cdrom.py
index 235f587..ee3c397 100644
--- a/repos/domain/install_linux_cdrom.py
+++ b/repos/domain/install_linux_cdrom.py
@@ -16,7 +16,7 @@ from libvirt import libvirtError
from src import sharedmod
from src import env_parser
from utils import utils
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('guestname', 'virt_type', 'guestos', 'guestarch',)
optional_params = ('uuid',
@@ -93,7 +93,7 @@ def prepare_boot_guest(domobj, dict, logger, installtype):
guestname = params['guestname']
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
domain = xmlobj.add_domain(params)
xmlobj.add_disk(params, domain)
@@ -258,7 +258,7 @@ def install_linux_cdrom(params):
logger.error("unknown virt type: %s" % virt_type)
return 1
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
guestxml = xmlobj.build_domain_install(params)
logger.debug('dump installation guest xml:\n%s' % guestxml)
diff --git a/repos/domain/install_linux_net.py b/repos/domain/install_linux_net.py
index 9841eed..105c0ea 100644
--- a/repos/domain/install_linux_net.py
+++ b/repos/domain/install_linux_net.py
@@ -16,7 +16,7 @@ from libvirt import libvirtError
from src import sharedmod
from src import env_parser
from utils import utils
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('guestname', 'virt_type', 'guestos', 'guestarch','netmethod',)
optional_params = ('uuid',
@@ -50,7 +50,7 @@ def prepare_boot_guest(domobj, dict, logger, installtype):
guestname = params['guestname']
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
domain = xmlobj.add_domain(params)
xmlobj.add_disk(params, domain)
@@ -240,7 +240,7 @@ def install_linux_net(params):
logger.error("unknown virt type %s" % virt_type)
# Prepare guest installation xml
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
guestxml = xmlobj.build_domain_install(params)
logger.debug('dump installation guest xml:\n%s' % guestxml)
diff --git a/repos/domain/install_windows_cdrom.py b/repos/domain/install_windows_cdrom.py
index 6cec08a..825aa29 100644
--- a/repos/domain/install_windows_cdrom.py
+++ b/repos/domain/install_windows_cdrom.py
@@ -15,7 +15,7 @@ from libvirt import libvirtError
from src import sharedmod
from src import env_parser
from utils import utils
-from utils import xmlbuilder
+from utils import xml_builder
VIRSH_QUIET_LIST = "virsh --quiet list --all|awk '{print $2}'|grep \"^%s$\""
VM_STAT = "virsh --quiet list --all| grep \"\\b%s\\b\"|grep off"
@@ -152,7 +152,7 @@ def prepare_boot_guest(domobj, dict, installtype):
guestname = params['guestname']
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
domain = xmlobj.add_domain(params)
xmlobj.add_disk(params, domain)
@@ -285,7 +285,7 @@ def install_windows_cdrom(params):
return 1
params['floppysource'] = FLOOPY_IMG
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
guestxml = xmlobj.build_domain_install_win(params)
logger.debug('dump installation guest xml:\n%s' % guestxml)
diff --git a/repos/domain/migrate.py b/repos/domain/migrate.py
index 2e6b40d..7a38a29 100644
--- a/repos/domain/migrate.py
+++ b/repos/domain/migrate.py
@@ -11,7 +11,7 @@ import libvirt
from libvirt import libvirtError
from src import sharedmod
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('transport',
'target_machine',
diff --git a/repos/domain/update_devflag.py b/repos/domain/update_devflag.py
index 9286918..6c75fd6 100644
--- a/repos/domain/update_devflag.py
+++ b/repos/domain/update_devflag.py
@@ -12,7 +12,7 @@ from libvirt import libvirtError
from src import sharedmod
from utils import utils
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('guestname', 'devtype', 'username', 'password',)
optional_params = ()
@@ -192,7 +192,7 @@ def update_devflag(params):
guestobj = minidom.parseString(guestxml)
# Generat device XML for original use
- origxmlobj = xmlbuilder.XmlBuilder()
+ origxmlobj = xml_builder.XmlBuilder()
if devtype == 'cdrom':
origxmlobj.add_cdrom(xmlargs, guestobj)
@@ -217,7 +217,7 @@ def update_devflag(params):
return 1
# Generate device XML for updating
- newxmlobj = xmlbuilder.XmlBuilder()
+ newxmlobj = xml_builder.XmlBuilder()
if devtype == 'cdrom':
xmlargs['bootcd'] = '/var/lib/libvirt/boot/cdrom-new.img'
diff --git a/repos/interface/create.py b/repos/interface/create.py
index 9df3464..3afcafb 100644
--- a/repos/interface/create.py
+++ b/repos/interface/create.py
@@ -10,7 +10,7 @@ from libvirt import libvirtError
from src import sharedmod
from utils import utils
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('ifacename',)
optional_params = ()
diff --git a/repos/interface/define.py b/repos/interface/define.py
index 630ae45..4d39e6a 100644
--- a/repos/interface/define.py
+++ b/repos/interface/define.py
@@ -8,7 +8,7 @@ import libvirt
from libvirt import libvirtError
from src import sharedmod
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('ifacename', 'ifacetype',)
optional_params = ()
@@ -37,7 +37,7 @@ def define(params):
logger.error("interface %s have been defined" % ifacename)
return 1
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
iface_xml = xmlobj.build_host_interface(params)
logger.debug("interface xml:\n%s" %iface_xml)
diff --git a/repos/interface/destroy.py b/repos/interface/destroy.py
index 7dc0320..4dcfc73 100644
--- a/repos/interface/destroy.py
+++ b/repos/interface/destroy.py
@@ -10,7 +10,7 @@ from libvirt import libvirtError
from src import sharedmod
from utils import utils
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('ifacename',)
optional_params = ()
diff --git a/repos/interface/undefine.py b/repos/interface/undefine.py
index 757d3f9..85d4489 100644
--- a/repos/interface/undefine.py
+++ b/repos/interface/undefine.py
@@ -8,7 +8,7 @@ import libvirt
from libvirt import libvirtError
from src import sharedmod
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('ifacename',)
optional_params = ()
diff --git a/repos/network/create.py b/repos/network/create.py
index d50c84d..8fc87a2 100644
--- a/repos/network/create.py
+++ b/repos/network/create.py
@@ -10,7 +10,7 @@ import libvirt
from libvirt import libvirtError
from src import sharedmod
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('networkname',
'bridgename',
@@ -46,7 +46,7 @@ def create(params):
logger.error("the %s network is running" % networkname)
return 1
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
netxml = xmlobj.build_network(params)
logger.debug("%s network xml:\n%s" % (networkname, netxml))
diff --git a/repos/network/define.py b/repos/network/define.py
index 8c02d2a..beafe1d 100644
--- a/repos/network/define.py
+++ b/repos/network/define.py
@@ -10,7 +10,7 @@ import libvirt
from libvirt import libvirtError
from src import sharedmod
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('networkname',
'bridgename',
@@ -49,7 +49,7 @@ def define(params):
logger.error("%s network is defined" % networkname)
return 1
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
netxml = xmlobj.build_network(params)
logger.debug("network xml:\n%s" % netxml)
diff --git a/repos/npiv/create_virtual_hba.py b/repos/npiv/create_virtual_hba.py
index a558fbc..a5005ca 100644
--- a/repos/npiv/create_virtual_hba.py
+++ b/repos/npiv/create_virtual_hba.py
@@ -6,7 +6,7 @@ import sys
import re
import commands
import xml.dom.minidom
-from utils import xmlbuilder
+from utils import xml_builder
import libvirt
from libvirt import libvirtError
@@ -70,7 +70,7 @@ def create_virtual_hba(params):
else:
logger.info("No NPIV capabilities on '%s'" % fc_name)
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
nodedev_xml = xmlobj.build_nodedev(params)
logger.debug("node device xml:\n%s" % nodedev_xml)
diff --git a/repos/regression/multiple_thread_block_on_domain_create.py b/repos/regression/multiple_thread_block_on_domain_create.py
index a72fdaf..1c85706 100644
--- a/repos/regression/multiple_thread_block_on_domain_create.py
+++ b/repos/regression/multiple_thread_block_on_domain_create.py
@@ -16,7 +16,7 @@ from libvirt import libvirtError
from src import sharedmod
from utils import utils
from utils import env_parser
-from utils import xmlbuilder
+from utils import xml_builder
IMAG_PATH = "/var/lib/libvirt/images/"
DISK_DD = "dd if=/dev/zero of=%s bs=1 count=1 seek=6G"
@@ -76,7 +76,7 @@ class guest_install(Thread):
else:
self.logger.info("creating disk images file is successful.")
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
guestxml = xmlobj.build_domain_install(guest_params)
self.logger.debug("guestxml is %s" % guestxml)
self.logger.info('create guest %sfrom xml description' % self.name)
diff --git a/repos/snapshot/internal_create.py b/repos/snapshot/internal_create.py
index d0f3906..6ddf350 100644
--- a/repos/snapshot/internal_create.py
+++ b/repos/snapshot/internal_create.py
@@ -11,7 +11,7 @@ from libvirt import libvirtError
from src import sharedmod
from utils import utils
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('guestname',)
optional_params = ('snapshotname',)
@@ -63,7 +63,7 @@ def internal_create(params):
logger.error("checking failed")
return 1
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
snapshot_xml = xmlobj.build_domain_snapshot(params)
logger.debug("%s snapshot xml:\n%s" % (guestname, snapshot_xml))
diff --git a/repos/storage/activate_pool.py b/repos/storage/activate_pool.py
index 3954457..51aa7fc 100644
--- a/repos/storage/activate_pool.py
+++ b/repos/storage/activate_pool.py
@@ -9,7 +9,7 @@ import libvirt
from libvirt import libvirtError
from src import sharedmod
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('poolname',)
optional_params = ()
diff --git a/repos/storage/create_dir_pool.py b/repos/storage/create_dir_pool.py
index 067ae92..5747585 100644
--- a/repos/storage/create_dir_pool.py
+++ b/repos/storage/create_dir_pool.py
@@ -9,7 +9,7 @@ import libvirt
from libvirt import libvirtError
from src import sharedmod
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('poolname', 'pooltype',)
optional_params = ()
@@ -45,7 +45,7 @@ def create_dir_pool(params):
logger.error("%s storage pool has already been created" % poolname)
return 1
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
poolxml = xmlobj.build_pool(params)
logger.debug("storage pool xml:\n%s" % poolxml)
diff --git a/repos/storage/create_dir_volume.py b/repos/storage/create_dir_volume.py
index 155eb3b..41fb270 100644
--- a/repos/storage/create_dir_volume.py
+++ b/repos/storage/create_dir_volume.py
@@ -11,7 +11,7 @@ import libvirt
from libvirt import libvirtError
from src import sharedmod
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('poolname', 'volname', 'volformat', 'capacity',)
optional_params = ()
@@ -112,7 +112,7 @@ def create_dir_volume(params):
the volume information in the pool %s" % poolname)
virsh_vol_list(poolname)
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
volumexml = xmlobj.build_volume(params)
logger.debug("volume xml:\n%s" % volumexml)
diff --git a/repos/storage/create_fs_pool.py b/repos/storage/create_fs_pool.py
index 7daf5f6..9bff2c7 100644
--- a/repos/storage/create_fs_pool.py
+++ b/repos/storage/create_fs_pool.py
@@ -9,8 +9,8 @@ import libvirt
from libvirt import libvirtError
from src import sharedmod
-from utils import xmlbuilder
-from utils import XMLParser
+from utils import xml_builder
+from utils import xml_parser
required_params = ('poolname', 'sourcepath', 'pooltype',)
optional_params = ('sourceformat',)
@@ -30,7 +30,7 @@ def check_pool_create_libvirt(conn, poolname, logger):
def check_pool_create_OS(poolobj, logger):
"""Check the result of create storage pool on OS level. """
poolxml = poolobj.XMLDesc(0)
- out = XMLParser.XMLParser().parse(poolxml)
+ out = xml_parser.xml_parser().parse(poolxml)
src_path = out["source"]["device"]["attr"]["path"]
dest_path = out["target"]["path"]
logger.info("src path: %s tgt path: %s" % (src_path, dest_path))
@@ -64,7 +64,7 @@ def create_fs_pool(params):
logger.error("%s storage pool has already been created" % poolname)
return 1
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
poolxml = xmlobj.build_pool(params)
logger.debug("storage pool xml:\n%s" % poolxml)
diff --git a/repos/storage/create_iscsi_pool.py b/repos/storage/create_iscsi_pool.py
index f108a59..85428cd 100644
--- a/repos/storage/create_iscsi_pool.py
+++ b/repos/storage/create_iscsi_pool.py
@@ -8,7 +8,7 @@ import libvirt
from libvirt import libvirtError
from src import sharedmod
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('poolname', 'sourcename', 'sourcepath', 'pooltype',)
optional_params = ()
@@ -44,7 +44,7 @@ def create_iscsi_pool(params):
logger.error("%s storage pool has already been created" % poolname)
return 1
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
poolxml = xmlobj.build_pool(params)
logger.debug("storage pool xml:\n%s" % poolxml)
diff --git a/repos/storage/create_logical_volume.py b/repos/storage/create_logical_volume.py
index 714480c..294ac73 100644
--- a/repos/storage/create_logical_volume.py
+++ b/repos/storage/create_logical_volume.py
@@ -12,7 +12,7 @@ from libvirt import libvirtError
from src import sharedmod
from utils import utils
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('poolname', 'pooltype', 'volname', 'capacity',)
optional_params = ()
@@ -101,7 +101,7 @@ def create_logical_volume(params):
params['volpath'] = "%s/%s" % (poolpath, volname)
logger.debug("volume target path: %s" % params['volpath'])
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
volxml = xmlobj.build_volume(params)
logger.debug("storage volume xml:\n%s" % volxml)
diff --git a/repos/storage/create_netfs_pool.py b/repos/storage/create_netfs_pool.py
index 2a0cd23..e16ea63 100644
--- a/repos/storage/create_netfs_pool.py
+++ b/repos/storage/create_netfs_pool.py
@@ -9,8 +9,8 @@ import libvirt
from libvirt import libvirtError
from src import sharedmod
-from utils import xmlbuilder
-from utils import XMLParser
+from utils import xml_builder
+from utils import xml_parser
required_params = ('poolname', 'sourcename', 'sourcepath', 'pooltype',)
optional_params = ('targetpath',)
@@ -36,7 +36,7 @@ def check_pool_create_OS(conn, poolname, logger):
poolobj = conn.storagePoolLookupByName(poolname)
poolxml = poolobj.XMLDesc(0)
# parse the xml to see where this is mounted...
- out = XMLParser.XMLParser().parse(poolxml)
+ out = xml_parser.xml_parser().parse(poolxml)
dest_path = out["target"]["path"]
src_host = out["source"]["host"]["attr"]["name"]
src_path = out["source"]["dir"]["attr"]["path"]
@@ -71,7 +71,7 @@ def create_netfs_pool(params):
logger.error("%s storage pool has already been created" % poolname)
return 1
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
poolxml = xmlobj.build_pool(params)
logger.debug("storage pool xml:\n%s" % poolxml)
diff --git a/repos/storage/create_netfs_volume.py b/repos/storage/create_netfs_volume.py
index e5f711a..da96f53 100644
--- a/repos/storage/create_netfs_volume.py
+++ b/repos/storage/create_netfs_volume.py
@@ -11,7 +11,7 @@ import libvirt
from libvirt import libvirtError
from src import sharedmod
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('poolname', 'volname', 'volformat', 'capacity',)
optional_params = ()
@@ -112,7 +112,7 @@ def create_netfs_volume(params):
ouput the volume information in the pool %s" % poolname)
virsh_vol_list(poolname)
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
volumexml = xmlobj.build_volume(params)
logger.debug("volume xml:\n%s" % volumexml)
diff --git a/repos/storage/create_partition_volume.py b/repos/storage/create_partition_volume.py
index 3ba802b..317697e 100644
--- a/repos/storage/create_partition_volume.py
+++ b/repos/storage/create_partition_volume.py
@@ -10,7 +10,7 @@ import libvirt
from libvirt import libvirtError
from src import sharedmod
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('poolname', 'volname', 'volformat', 'capacity',)
optional_params = ()
@@ -82,7 +82,7 @@ def create_partition_volume(params):
ouput the volume information in the pool %s" % poolname)
virsh_vol_list(poolname)
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
volumexml = xmlobj.build_volume(params)
logger.debug("volume xml:\n%s" % volumexml)
diff --git a/repos/storage/define_dir_pool.py b/repos/storage/define_dir_pool.py
index b45767d..0c8dbb7 100644
--- a/repos/storage/define_dir_pool.py
+++ b/repos/storage/define_dir_pool.py
@@ -10,7 +10,7 @@ import libvirt
from libvirt import libvirtError
from src import sharedmod
-from utils import xmlbuilder
+from utils import xml_builder
VIRSH_POOLLIST = "virsh --quiet pool-list --all|awk '{print $1}'|grep \"^%s$\""
POOL_STAT = "virsh --quiet pool-list --all|grep \"^%s\\b\" |grep \"inactive\""
@@ -53,7 +53,7 @@ def define_dir_pool(params):
logger.error("%s storage pool is defined" % poolname)
return 1
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
poolxml = xmlobj.build_pool(params)
logger.debug("storage pool xml:\n%s" % poolxml)
diff --git a/repos/storage/define_disk_pool.py b/repos/storage/define_disk_pool.py
index 619292f..56a2f09 100644
--- a/repos/storage/define_disk_pool.py
+++ b/repos/storage/define_disk_pool.py
@@ -9,7 +9,7 @@ import libvirt
from libvirt import libvirtError
from src import sharedmod
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('poolname', 'pooltype', 'sourcepath',)
optional_params = ('sourceformat', 'targetpath',)
@@ -54,7 +54,7 @@ def define_disk_pool(params):
logger.error("%s storage pool is defined" % poolname)
return 1
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
poolxml = xmlobj.build_pool(params)
logger.debug("storage pool xml:\n%s" % poolxml)
diff --git a/repos/storage/define_iscsi_pool.py b/repos/storage/define_iscsi_pool.py
index fdb09a2..1a1e247 100644
--- a/repos/storage/define_iscsi_pool.py
+++ b/repos/storage/define_iscsi_pool.py
@@ -9,7 +9,7 @@ import libvirt
from libvirt import libvirtError
from src import sharedmod
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('poolname', 'pooltype', 'sourcename', 'sourcepath',)
optional_params = ()
@@ -57,7 +57,7 @@ def define_iscsi_pool(params):
logger.error("%s storage pool is ALREADY defined" % poolname)
return 1
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
poolxml = xmlobj.build_pool(params)
logger.debug("storage pool xml:\n%s" % poolxml)
diff --git a/repos/storage/define_logical_pool.py b/repos/storage/define_logical_pool.py
index ebbdc76..869f2c5 100644
--- a/repos/storage/define_logical_pool.py
+++ b/repos/storage/define_logical_pool.py
@@ -9,7 +9,7 @@ import libvirt
from libvirt import libvirtError
from src import sharedmod
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('poolname', 'pooltype', 'sourcename', 'sourcepath',)
optional_params = ()
@@ -42,7 +42,7 @@ def define_logical_pool(params):
logger.error("%s storage pool is defined" % poolname)
return 1
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
poolxml = xmlobj.build_pool(params)
logger.debug("storage pool xml:\n%s" % poolxml)
diff --git a/repos/storage/define_mpath_pool.py b/repos/storage/define_mpath_pool.py
index 930f9b8..0780706 100644
--- a/repos/storage/define_mpath_pool.py
+++ b/repos/storage/define_mpath_pool.py
@@ -8,7 +8,7 @@ import libvirt
from libvirt import libvirtError
from src import sharedmod
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('poolname', 'pooltype',)
optional_params = ('targetpath',)
@@ -50,7 +50,7 @@ def define_mpath_pool(params):
logger.error("%s storage pool is defined" % poolname)
return 1
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
poolxml = xmlobj.build_pool(params)
logger.debug("storage pool xml:\n%s" % poolxml)
diff --git a/repos/storage/define_netfs_pool.py b/repos/storage/define_netfs_pool.py
index e607da2..d0fdf12 100644
--- a/repos/storage/define_netfs_pool.py
+++ b/repos/storage/define_netfs_pool.py
@@ -9,7 +9,7 @@ import libvirt
from libvirt import libvirtError
from src import sharedmod
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('poolname', 'pooltype', 'sourcename', 'sourcepath',)
optional_params = ('targetpath',)
@@ -42,7 +42,7 @@ def define_netfs_pool(params):
logger.error("%s storage pool is defined" % poolname)
return 1
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
poolxml = xmlobj.build_pool(params)
logger.debug("storage pool xml:\n%s" % poolxml)
diff --git a/repos/storage/define_scsi_pool.py b/repos/storage/define_scsi_pool.py
index 8be591a..c486a95 100644
--- a/repos/storage/define_scsi_pool.py
+++ b/repos/storage/define_scsi_pool.py
@@ -9,7 +9,7 @@ import libvirt
from libvirt import libvirtError
from src import sharedmod
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('poolname', 'pooltype', 'sourcename',)
optional_params = ('targetpath',)
@@ -54,7 +54,7 @@ def define_scsi_pool(params):
logger.error("%s storage pool is defined" % poolname)
return 1
- xmlobj = xmlbuilder.XmlBuilder()
+ xmlobj = xml_builder.XmlBuilder()
poolxml = xmlobj.build_pool(params)
logger.debug("storage pool xml:\n%s" % poolxml)
diff --git a/repos/storage/delete_partition_volume.py b/repos/storage/delete_partition_volume.py
index b70ff45..17a4cd9 100644
--- a/repos/storage/delete_partition_volume.py
+++ b/repos/storage/delete_partition_volume.py
@@ -9,7 +9,7 @@ import libvirt
from libvirt import libvirtError
from src import sharedmod
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('poolname', 'volname',)
optional_params = ()
diff --git a/repos/storage/destroy_pool.py b/repos/storage/destroy_pool.py
index 304f1bb..8566a84 100644
--- a/repos/storage/destroy_pool.py
+++ b/repos/storage/destroy_pool.py
@@ -8,7 +8,7 @@ import libvirt
from libvirt import libvirtError
from src import sharedmod
-from utils import xmlbuilder
+from utils import xml_builder
required_params = ('poolname',)
optional_params = ()
diff --git a/src/logxmlparser.py b/src/logxmlparser.py
index 3b377ce..36d743e 100644
--- a/src/logxmlparser.py
+++ b/src/logxmlparser.py
@@ -24,7 +24,7 @@ from xml.dom.minidom import Document
from src import exception
-class LogXMLParser(object):
+class Logxml_parser(object):
""" Generate and parser log xml file
"""
def __init__(self, logxml):
diff --git a/utils/xmlbuilder.py b/utils/xml_builder.py
similarity index 91%
rename from utils/xmlbuilder.py
rename to utils/xml_builder.py
index 85cda03..435aa84 100644
--- a/utils/xmlbuilder.py
+++ b/utils/xml_builder.py
@@ -1,6 +1,6 @@
#!/usr/bin/env python
#
-# xmlbuilder.py: Class for building XML for libvirt objects.
+# xml_builder.py: Class for building XML for libvirt objects.
#
# Copyright (C) 2010-2012 Red Hat, Inc.
#
@@ -21,7 +21,7 @@ __DEBUG__ = False
import os, sys
import xml.dom.minidom
-import xmlgenerator
+import xml_generator
class XmlBuilder:
"""Operation for building domain xml"""
@@ -29,43 +29,43 @@ class XmlBuilder:
print doc.toprettyxml()
def add_domain(self, params):
- domain = xmlgenerator.domain_xml(params)
+ domain = xml_generator.domain_xml(params)
return domain
def add_disk(self, params, domain):
- disk = xmlgenerator.disk_xml(params)
+ disk = xml_generator.disk_xml(params)
disk_node = domain.importNode(disk.childNodes[0], True)
domain.getElementsByTagName("devices")[0].insertBefore(
disk_node, domain.getElementsByTagName("console")[0])
def add_floppy(self, params, domain):
- floppy = xmlgenerator.floppy_xml(params)
+ floppy = xml_generator.floppy_xml(params)
floppy_node = domain.importNode(floppy.childNodes[0], True)
domain.getElementsByTagName("devices")[0].insertBefore(
floppy_node, domain.getElementsByTagName("console")[0])
def add_cdrom(self, params, domain):
- cdrom = xmlgenerator.disk_xml(params, True)
+ cdrom = xml_generator.disk_xml(params, True)
cdrom_node = domain.importNode(cdrom.childNodes[0], True)
domain.getElementsByTagName("devices")[0].insertBefore(
cdrom_node, domain.getElementsByTagName("console")[0])
def add_interface(self, params, domain):
- interface = xmlgenerator.interface_xml(params)
+ interface = xml_generator.interface_xml(params)
interface_node = domain.importNode(interface.childNodes[0], True)
domain.getElementsByTagName("devices")[0].insertBefore(
interface_node, domain.getElementsByTagName("console")[0])
return interface
def add_hostdev(self, params, domain):
- hostdev = xmlgenerator.hostdev_xml(params)
+ hostdev = xml_generator.hostdev_xml(params)
hostdev_node = domain.importNode(hostdev.childNodes[0], True)
domain.getElementsByTagName("devices")[0].insertBefore(
hostdev_node, domain.getElementsByTagName("console")[0])
return hostdev
def build_domain_install(self, params):
- domain = xmlgenerator.domain_xml(params, True)
+ domain = xml_generator.domain_xml(params, True)
self.add_disk(params, domain)
if params['virt_type'] != 'xenpv':
if params.has_key('bootcd'):
@@ -76,7 +76,7 @@ class XmlBuilder:
return domain.toxml()
def build_domain_install_win(self, params):
- domain = xmlgenerator.domain_xml(params, True)
+ domain = xml_generator.domain_xml(params, True)
self.add_disk(params, domain)
self.add_floppy(params, domain)
if params.has_key('bootcd'):
@@ -102,7 +102,7 @@ class XmlBuilder:
else:
print 'Wrong harddisk model.'
- disk = xmlgenerator.disk_xml(params)
+ disk = xml_generator.disk_xml(params)
if params['virt_type'] == 'xenpv':
disk.getElementsByTagName("target")[0].setAttribute("dev", "xvdb")
else:
@@ -123,7 +123,7 @@ class XmlBuilder:
else:
print 'Wrong cdrom model.'
- cdrom = xmlgenerator.disk_xml(params, True)
+ cdrom = xml_generator.disk_xml(params, True)
if params['virt_type'] == 'xenpv':
cdrom.getElementsByTagName("target")[0].setAttribute("dev", "xvdc")
else:
@@ -134,61 +134,61 @@ class XmlBuilder:
return cdrom.toxml()
def build_floppy(self, params):
- floppy = xmlgenerator.floppy_xml(params)
+ floppy = xml_generator.floppy_xml(params)
if __DEBUG__:
self.write_toxml(floppy)
return floppy.toxml()
def build_interface(self, params):
- interface = xmlgenerator.interface_xml(params)
+ interface = xml_generator.interface_xml(params)
if __DEBUG__:
self.write_toxml(interface)
return interface.toxml()
def build_hostdev(self, params):
- hostdev = xmlgenerator.hostdev_xml(params)
+ hostdev = xml_generator.hostdev_xml(params)
if __DEBUG__:
self.write_toxml(hostdev)
return hostdev.toxml()
def build_pool(self, params):
- pool = xmlgenerator.pool_xml(params)
+ pool = xml_generator.pool_xml(params)
if __DEBUG__:
self.write_toxml(pool)
return pool.toxml()
def build_volume(self, params):
- volume = xmlgenerator.volume_xml(params)
+ volume = xml_generator.volume_xml(params)
if __DEBUG__:
self.write_toxml(volume)
return volume.toxml()
def build_network(self, params):
- network = xmlgenerator.network_xml(params)
+ network = xml_generator.network_xml(params)
if __DEBUG__:
self.write_toxml(network)
return network.toxml()
def build_host_interface(self, params):
- interface = xmlgenerator.host_iface_xml(params)
+ interface = xml_generator.host_iface_xml(params)
if __DEBUG__:
self.write_toxml(interface)
return interface.toxml()
def build_domain_snapshot(self, params):
- snapshot = xmlgenerator.snapshot_xml(params)
+ snapshot = xml_generator.snapshot_xml(params)
if __DEBUG__:
self.write_toxml(snapshot)
return snapshot.toxml()
def build_secret(self, params):
- secret = xmlgenerator.secret_xml(params)
+ secret = xml_generator.secret_xml(params)
if __DEBUG__:
self.write_toxml(secret)
return secret.toxml()
def build_nodedev(self, params):
- nodedev = xmlgenerator.nodedev_xml(params)
+ nodedev = xml_generator.nodedev_xml(params)
if __DEBUG__:
self.write_toxml(nodedev)
return nodedev.toxml()
diff --git a/utils/xmlgenerator.py b/utils/xml_generator.py
similarity index 99%
rename from utils/xmlgenerator.py
rename to utils/xml_generator.py
index c2bdec5..e173543 100644
--- a/utils/xmlgenerator.py
+++ b/utils/xml_generator.py
@@ -1,6 +1,6 @@
#!/usr/bin/env python
#
-# xmlgenerator.py: Generate XML for libvirt objects.
+# xml_generator.py: Generate XML for libvirt objects.
#
# Copyright (C) 2010-2012 Red Hat, Inc.
#
diff --git a/utils/XMLParser.py b/utils/xml_parser.py
similarity index 96%
rename from utils/XMLParser.py
rename to utils/xml_parser.py
index 20a376e..04e7501 100644
--- a/utils/XMLParser.py
+++ b/utils/xml_parser.py
@@ -1,6 +1,6 @@
#!/usr/bin/env python
#
-# XMLParser.py: Parse XML document, the result is a python dict.
+# xml_parser.py: Parse XML document, the result is a python dict.
#
# Copyright (C) 2010-2012 Red Hat, Inc.
#
@@ -21,8 +21,8 @@ import os
from xml.dom import minidom
import StringIO
-class XMLParser(object):
- """Class XMLParser. It parses and xml document into a python dictionary.
+class xml_parser(object):
+ """Class xml_parser. It parses and xml document into a python dictionary.
The elements of the xml documents will be python dictionary keys. For
example, the xml document:
<firstel>firstdata</firstel>
@@ -32,7 +32,7 @@ class XMLParser(object):
will be parsed into the python dictionary:
{ "firstel":"firstdata" , "secondel":{"subsinsecond":"seconddata"} }
Then the data can be retrieve as:
- out = XMLParser.XMLParser().parse(xml)
+ out = xml_parser.xml_parser().parse(xml)
out["firstel"] (this will be firstdata )
out["secondel"]["subinsecond"] (this will be seconddata)