[Libvirt-cim] [PATCH] [TEST] Adding new tc to verify fs storage pool creation

Deepti B. Kalakeri deeptik at linux.vnet.ibm.com
Fri Jun 26 22:31:04 UTC 2009


# HG changeset patch
# User Deepti B. Kalakeri<deeptik at linux.vnet.ibm.com>
# Date 1246055400 25200
# Node ID 1e14cb463dc2251286e8e82099ac410c4e8d3453
# Parent  d67a606da9f7d631368d04280865eb9a21e7ea8a
[TEST] Adding new tc to verify fs storage pool creation.

This tc will not be run in the batch mode of cimtest and hence needs to
be run individually using the command below.

python create_verify_storagepool.py -t 2  -d /dev/sda4 -m /tmp/mnt -n diskfs
         -v Xen -u <username> -p <passwd>

Tested with Xen on RHEL with current sources for fs type pool.
Will Update the patch to include logical pool verification as well.
Signed-off-by: Deepti B. Kalakeri <deeptik at linux.vnet.ibm.com>

diff -r d67a606da9f7 -r 1e14cb463dc2 suites/libvirt-cim/misc_cimtests/create_verify_storagepool.py
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/suites/libvirt-cim/misc_cimtests/create_verify_storagepool.py	Fri Jun 26 15:30:00 2009 -0700
@@ -0,0 +1,235 @@
+#!/usr/bin/python
+#
+# Copyright 2009 IBM Corp.
+#
+# Authors:
+#    Deepti B. Kalakeri<dkalaker at in.ibm.com> 
+#    
+#
+# This library is free software; you can redistribute it and/or
+# modify it under the terms of the GNU General Public
+# License as published by the Free Software Foundation; either
+# version 2.1 of the License, or (at your option) any later version.
+#
+# This library is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+# General Public License for more details.
+#
+# You should have received a copy of the GNU General Public
+# License along with this library; if not, write to the Free Software
+# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307  USA
+#
+# This test case should test the  CreateChildResourcePool service 
+# supplied by the RPCS provider. 
+# This tc verifies the FileSystem Type storage pool.
+#
+# The test case is not run in the batch run and we need to run it using 
+# the following command:
+# python create_verify_storagepool.py -t 2  -d /dev/sda4 -m /tmp/mnt -n diskfs 
+#         -v Xen -u <username> -p <passwd>
+# 
+# Where t can be :
+#       2 - FileSystem
+#       4 - Logical etc
+# 
+# 
+#                                                         Date : 27.06.2009
+
+import os
+import sys
+import TestSuite
+from optparse import OptionParser
+from CimTest import Globals
+from XenKvmLib.classes import inst_to_mof, get_typed_class
+from CimTest.Globals import logger, log_param
+from commands  import getstatusoutput
+from pywbem import WBEMConnection, cim_types
+from distutils.text_file import TextFile
+from XenKvmLib.pool import get_pool_rasds
+
+PASS = 0
+FAIL = 1
+supp_types = [ 'Xen', 'KVM' ]
+pool_types = { 2 : 'DISK_POOL_FS', 4 : 'DISK_POOL_LOGICAL' }
+
+def verify_inputs(part_dev, mount_pt, pool_type):
+    if pool_type not in pool_types.keys():
+        logger.error("Pool type '%s' specified is not valid", pool_type)
+        return FAIL
+     
+    # TextFile skips all the lines which have comments and which are blank
+    fd = TextFile(filename="/etc/fstab")
+    fstab_info = [x.rstrip('\n') for x in fd.readlines()]
+    fd.close()
+
+    for line in fstab_info:
+        try:
+            # Check if the specified partition is mounted before using it 
+            if part_dev in line.split()[0]:
+                logger.error("[%s] already mounted", part_dev)
+                raise Exception("Please specify free partition other than " \
+                                "[%s]" % part_dev)
+
+            # Check if mount point is already used for mounting
+            if mount_pt in line.split()[1]:
+                logger.error("[%s] already mounted", mount_pt)
+                raise Exception("Please specify dir other than [%s]" %mount_pt)
+
+        except Exception, details:
+            logger.error("Exception details is %s", details)
+            return FAIL
+   
+    ## Check if the mount point specified already exist if not then create it..
+    if not os.path.exists(mount_pt):
+        os.mkdir(mount_pt)
+    else:
+        ## Check if the mount point specified is a dir
+        if not os.path.isdir(mount_pt):
+            logger.error("The mount point [%s] should be a dir", mount_pt)
+            return FAIL
+
+        files = os.listdir(mount_pt)
+        if len(files) != 0:
+            logger.info("The mount point [%s] given is not empty", mount_pt)
+
+    return PASS
+
+def get_uri(virt):
+    if virt == 'Xen':
+        vuri = 'xen:///'
+    elif vir_type == 'Kvm':
+        vuri = 'qemu:///system'
+    return vuri
+
+def verify_pool(virt, pool_name):
+    vuri = get_uri(virt)
+    cmd = "virsh -c %s pool-list --all | grep %s" %(vuri, pool_name)
+    return getstatusoutput(cmd)
+
+def main():
+    usage = "usage: %prog [options] \nex: %prog -i localhost"
+    parser = OptionParser(usage)
+
+    parser.add_option("-i", "--host-url", dest="h_url", default="localhost:5988",
+                      help="URL of CIMOM to connect to (host:port)")
+    parser.add_option("-N", "--ns", dest="ns", default="root/virt",
+                      help="Namespace (default is root/virt)")
+    parser.add_option("-u", "--user", dest="username", default=None,
+                      help="Auth username for CIMOM on source system")
+    parser.add_option("-p", "--pass", dest="password", default=None,
+                      help="Auth password for CIMOM on source system")
+    parser.add_option("-v", "--virt-type", dest="virt", default=None,
+                      help="Virtualization type [ Xen | KVM ]")
+    parser.add_option("-t", "--pool-type", dest="pool_type", default=None,
+                      help="Pool type:[ fs | logical ]")
+    parser.add_option("-d", "--part-dev", dest="part_dev", default=None,
+                      help="specify the free partition to be used")
+    parser.add_option("-m", "--mnt_pt", dest="mnt_pt", default=None, 
+                      help="Mount point to be used")
+    parser.add_option("-n", "--pool-name", dest="pool_name", default=None, 
+                      help="Pool to be created")
+
+    (options, args) = parser.parse_args()
+
+    try: 
+        if options.part_dev == None:
+            raise Exception("Free Partition to be mounted not specified")
+
+        if options.mnt_pt == None:
+            raise Exception("Mount points to be used not specified")
+
+        if options.pool_name == None:
+            raise Exception("Must specify the Pool Name to be created")
+
+        if options.virt == None or options.virt not in supp_types:
+            raise Exception("Must specify virtualization type")
+
+        if options.pool_type == None:
+            raise Exception("Must specify pool type to be tested")
+
+    except Exception, details:
+        print "FATAL: ", details
+        print parser.print_help()
+        return FAIL
+    
+    part_dev = options.part_dev
+    mount_pt = options.mnt_pt
+    pool_name = options.pool_name
+    pool_type = cim_types.Uint16(options.pool_type)
+    virt = options.virt
+    
+    testsuite = TestSuite.TestSuite(log=True)
+    log_param(file_name="cimtest.log")
+    print "Please check cimtest.log in the curr dir for debug log msgs..."
+    status = verify_inputs(part_dev, mount_pt, pool_type)
+    if status != PASS:
+        logger.error("Input verification failed")
+        return status
+   
+    status, out = verify_pool(virt, pool_name)
+    if status == PASS:
+        logger.error("Pool --> '%s' already exist", pool_name)
+        logger.error("Specify some other pool name")
+        return status
+
+    if ":" in options.h_url:
+        (sysname, port) = options.h_url.split(":")
+    else:
+        sysname = options.h_url
+
+    src_conn = WBEMConnection('http://%s' % sysname, 
+                              (options.username, options.password), options.ns)
+
+    os.environ['CIM_NS'] = Globals.CIM_NS = options.ns
+    os.environ['CIM_USER'] = Globals.CIM_USER = options.username
+    os.environ['CIM_PASS'] = Globals.CIM_PASS = options.password
+
+    try:
+        status,dp_rasds = get_pool_rasds(sysname, virt, "DiskPool")
+        for i in range(0, len(dp_rasds)):
+            pool_id = "%s/%s" %("DiskPool", pool_name)
+            dpool_rasd=dp_rasds[i]
+            dpool_rasd['PoolID'] = pool_id 
+            if dpool_rasd['Type'] == pool_type and \
+               dpool_rasd['InstanceID'] == 'Default':
+                dpool_rasd['DevicePaths'] =[part_dev]
+                dpool_rasd['Path'] = mount_pt
+                dpool_rasd['InstanceID'] = pool_id
+                break
+        pool_settings = inst_to_mof(dpool_rasd)
+        rpcs_cn = get_typed_class(virt, "ResourcePoolConfigurationService")
+        res = src_conn.InvokeMethod("CreateChildResourcePool",
+                                    rpcs_cn,
+                                    Settings=[pool_settings],
+                                    ElementName=pool_name)
+    except Exception, details:
+        logger.error("Exception details: %s", details)
+
+    status, out = verify_pool(virt, pool_name)
+    if status != PASS:
+        logger.error("Failed to create pool: %s ", pool_name)
+        return status 
+
+    vuri = get_uri(virt)
+    virsh = "virsh -c %s" % vuri
+    cmd = "%s pool-destroy  %s && %s pool-undefine %s" \
+           %(virsh, pool_name, virsh, pool_name)
+    ret, out = getstatusoutput(cmd)
+    if ret != PASS:
+        logger.info("WARNING: pool %s was not cleaned on %s", 
+                    pool_name, sysname)
+        logger.info("WARNING: Please remove it manually")
+
+    logger.info("Pool %s was successfully verified for pool type %s", 
+                pool_name , pool_types[pool_type])
+
+    # Place holder to give a hint to the user the tc passed 
+    # otherwise the user will have to look into the cimtest.log in the 
+    # current dir.
+    print "Pool '", pool_name,"' was successfully verified for pool type '", \
+          pool_types[pool_type] , "'"
+    return status
+if __name__=="__main__":
+    sys.exit(main())
+




More information about the Libvirt-cim mailing list