[Libvirt-cim] [PATCH] [TEST] #2 Remove revision branches

Kaitlin Rupert kaitlin at linux.vnet.ibm.com
Tue Jun 17 03:33:35 UTC 2008


# HG changeset patch
# User Kaitlin Rupert <karupert at us.ibm.com>
# Date 1213121071 25200
# Node ID e1f221334033e17120e32dc5ffa00c951b8ada78
# Parent  6652ca49942c0314ec1b2b1da504810325097250
[TEST] #2 Remove revision branches.

These brances were needed to distinguish the behavior between the 0.3.0 release and the current head of the mercurial tree.  Now that the cimtest tree has been tagged, cimtest release_0_1 should correspond with the functionality in libvirt-cim 0.4.0.

Updates:
 -Rebased on current tree
 -Also update RAFP 02

Signed-off-by: Kaitlin Rupert <karupert at us.ibm.com>

diff -r 6652ca49942c -r e1f221334033 suites/libvirt-cim/cimtest/AllocationCapabilities/02_alloccap_gi_errs.py
--- a/suites/libvirt-cim/cimtest/AllocationCapabilities/02_alloccap_gi_errs.py	Thu Jun 12 17:13:23 2008 +0800
+++ b/suites/libvirt-cim/cimtest/AllocationCapabilities/02_alloccap_gi_errs.py	Tue Jun 10 11:04:31 2008 -0700
@@ -62,14 +62,12 @@
 from XenKvmLib.test_doms import create_vnet
 from CimTest.Globals import do_main, platform_sup
 from XenKvmLib.classes import get_typed_class
-from XenKvmLib.const import CIM_REV
 from XenKvmLib.common_util import cleanup_restore, test_dpath, \
 create_diskpool_file
 
 diskid = "%s/%s" % ("DiskPool", test_dpath)
 memid = "%s/%s" % ("MemoryPool", 0)
 procid = "%s/%s" % ("ProcessorPool", 0)
-rev = 463
 
 sup_types = ['Xen', 'KVM', 'XenFV', 'LXC']
 @do_main(sup_types)     
@@ -112,8 +110,6 @@
                                   'desc' : 'No InstanceID specified' },
             "invalid_keyvalue" : { 'rc' : pywbem.CIM_ERR_NOT_FOUND,
                                    'desc' : 'Instance not found' }}
-    if CIM_REV < rev:
-        exp['invalid_keyvalue']['desc'] = 'Object could not be found'
 
     ret_value = try_getinstance(conn, classname, keys, field_name=field,
                                 expr_values=exp['invalid_keyvalue'], bug_no="")
diff -r 6652ca49942c -r e1f221334033 suites/libvirt-cim/cimtest/ElementAllocatedFromPool/03_reverse_errs.py
--- a/suites/libvirt-cim/cimtest/ElementAllocatedFromPool/03_reverse_errs.py	Thu Jun 12 17:13:23 2008 +0800
+++ b/suites/libvirt-cim/cimtest/ElementAllocatedFromPool/03_reverse_errs.py	Tue Jun 10 11:04:31 2008 -0700
@@ -34,7 +34,6 @@
 from CimTest import Globals
 from XenKvmLib.test_doms import destroy_and_undefine_all
 from XenKvmLib.common_util import try_assoc
-from XenKvmLib.const import CIM_REV
 from CimTest.ReturnCodes import PASS, FAIL	
 from CimTest.Globals import do_main, platform_sup, logger
 from XenKvmLib.vxml import get_class
@@ -60,7 +59,6 @@
                                         'desc'  : 'No such instance'
                                     } 
               }
-libvirt_rev = 393
 
 def err_invalid_ccname():
 # This is used to verify the that the  
@@ -157,10 +155,6 @@
               get_typed_class(virt, "ProcessorPool")
              ]
 
-    if CIM_REV < libvirt_rev:
-        expr_values['invalid_keyvalue']['desc'] = 'Invalid InstanceID or unsupported pool type'
-        expr_values['invalid_keyvalue']['rc'] = pywbem.CIM_ERR_FAILED
-
     for classname in sorted(lelist):
         keys = { "InstanceID" : "InvalidKeyValue" }
         field = "InstanceID_KeyValue"    
diff -r 6652ca49942c -r e1f221334033 suites/libvirt-cim/cimtest/ElementCapabilities/01_forward.py
--- a/suites/libvirt-cim/cimtest/ElementCapabilities/01_forward.py	Thu Jun 12 17:13:23 2008 +0800
+++ b/suites/libvirt-cim/cimtest/ElementCapabilities/01_forward.py	Tue Jun 10 11:04:31 2008 -0700
@@ -30,11 +30,9 @@
 from CimTest.Globals import do_main, logger, CIM_ERROR_ASSOCIATORNAMES, \
 CIM_ERROR_ENUMERATE
 from CimTest.ReturnCodes import PASS, FAIL, SKIP
-from XenKvmLib.const import CIM_REV
 from XenKvmLib.enumclass import enumerate
 
 sup_types = ['Xen', 'XenFV', 'KVM', 'LXC']
-ac_to_pool_version = 561
 
 def append_to_list(server, virt, poolname, valid_elc_id):
     keys_list = ['InstanceID']
@@ -87,11 +85,10 @@
     valid_elc_id = ["ManagementCapabilities", 
                     "MigrationCapabilities"]
 
-    if CIM_REV >= ac_to_pool_version:
-        valid_elc_name.append(get_typed_class(virt, "AllocationCapabilities"))
-        status, valid_elc_id = set_pool_info(server, virt, valid_elc_id)
-        if status != PASS:
-            return status
+    valid_elc_name.append(get_typed_class(virt, "AllocationCapabilities"))
+    status, valid_elc_id = set_pool_info(server, virt, valid_elc_id)
+    if status != PASS:
+        return status
 
     if len(elc) == 0:
         logger.error("ElementCapabilities association failed, excepted at least one instance")
diff -r 6652ca49942c -r e1f221334033 suites/libvirt-cim/cimtest/HostSystem/02_hostsystem_to_rasd.py
--- a/suites/libvirt-cim/cimtest/HostSystem/02_hostsystem_to_rasd.py	Thu Jun 12 17:13:23 2008 +0800
+++ b/suites/libvirt-cim/cimtest/HostSystem/02_hostsystem_to_rasd.py	Tue Jun 10 11:04:31 2008 -0700
@@ -58,7 +58,6 @@
 from CimTest.ReturnCodes import PASS, FAIL
 from XenKvmLib.rasd import verify_procrasd_values, verify_netrasd_values, \
 verify_diskrasd_values, verify_memrasd_values, rasd_init_list
-from XenKvmLib.const import CIM_REV
 
 sup_types = ['Xen', 'KVM', 'XenFV']
 
@@ -67,8 +66,6 @@
 test_vcpus  = 1
 test_mem    = 128
 test_mac    = "00:11:22:33:44:aa"
-rev = 529
-proc_instid_rev = 590
 
 def setup_env(server, virt="Xen"):
     vsxml_info = None
diff -r 6652ca49942c -r e1f221334033 suites/libvirt-cim/cimtest/HostSystem/03_hs_to_settdefcap.py
--- a/suites/libvirt-cim/cimtest/HostSystem/03_hs_to_settdefcap.py	Thu Jun 12 17:13:23 2008 +0800
+++ b/suites/libvirt-cim/cimtest/HostSystem/03_hs_to_settdefcap.py	Tue Jun 10 11:04:31 2008 -0700
@@ -49,12 +49,10 @@
 from CimTest.ReturnCodes import PASS, FAIL
 from XenKvmLib.test_xml import testxml
 from XenKvmLib.test_doms import destroy_and_undefine_all
-from XenKvmLib.const import CIM_REV
 
 sup_types = ['Xen', 'KVM', 'XenFV']
 test_dom = "domgst"
 test_vcpus = 1
-libvirtcim_sdc_rasd_rev = 571
 
 def setup_env(server, virt="Xen"):
     status = PASS
@@ -243,26 +241,6 @@
             logger.error("In ResourceType for %s " % rt)
             return FAIL
  
-        # The following properties have been removed in the patchset 571
-        # but is present in the rpm libvirt-cim and hence retained it.
-
-        if CIM_REV < libvirtcim_sdc_rasd_rev:
-            ppolicy = inst['PropertyPolicy']
-            if ppolicy != 0 and ppolicy != 1:
-                logger.error("In PropertyPolicy for %s " % ppolicy)
-                return FAIL
-
-            vrole  = inst['ValueRole']
-            if vrole < 0 or vrole > 4:
-                logger.error("In ValueRole %s " % vrole)
-                return FAIL
-
-            insid  = inst['InstanceID']
-            vrange = rangelist[insid]
-            if vrange != inst['ValueRange']:
-                logger.error("In ValueRange for %s " % vrange)
-                return FAIL
-
     except Exception, detail:
         logger.error("Error checking RASD attribute values %s" % detail)
         return FAIL
diff -r 6652ca49942c -r e1f221334033 suites/libvirt-cim/cimtest/Memory/01_memory.py
--- a/suites/libvirt-cim/cimtest/Memory/01_memory.py	Thu Jun 12 17:13:23 2008 +0800
+++ b/suites/libvirt-cim/cimtest/Memory/01_memory.py	Tue Jun 10 11:04:31 2008 -0700
@@ -31,13 +31,11 @@
 from XenKvmLib.vxml import XenXML, KVMXML, get_class
 from CimTest.Globals import logger
 from CimTest.Globals import do_main
-from XenKvmLib.const import CIM_REV
 
 sup_types = ['Xen', 'KVM', 'XenFV', 'LXC']
 
 test_dom = "test_domain"
 mem = 256 #MB
-mem_change_version=585
 
 @do_main(sup_types)
 def main():
@@ -45,10 +43,7 @@
     
     vsxml = get_class(options.virt)(test_dom, mem)
     vsxml.define(options.ip)
-    if CIM_REV >= mem_change_version: 
-        alloc_mem = int(vsxml.xml_get_mem())
-    else:
-        alloc_mem = int(vsxml.xml_get_mem())/1024
+    alloc_mem = int(vsxml.xml_get_mem())
     
     devid = "%s/mem" % test_dom
     key_list = { 'DeviceID' : devid,
diff -r 6652ca49942c -r e1f221334033 suites/libvirt-cim/cimtest/RASD/01_verify_rasd_fields.py
--- a/suites/libvirt-cim/cimtest/RASD/01_verify_rasd_fields.py	Thu Jun 12 17:13:23 2008 +0800
+++ b/suites/libvirt-cim/cimtest/RASD/01_verify_rasd_fields.py	Tue Jun 10 11:04:31 2008 -0700
@@ -54,7 +54,6 @@
 from XenKvmLib import vxml 
 from XenKvmLib.classes import get_typed_class
 from XenKvmLib import rasd 
-from XenKvmLib.const import CIM_REV
 from CimTest.Globals import logger
 from CimTest.ReturnCodes import PASS, FAIL
 from XenKvmLib import rasd
diff -r 6652ca49942c -r e1f221334033 suites/libvirt-cim/cimtest/RASD/02_enum.py
--- a/suites/libvirt-cim/cimtest/RASD/02_enum.py	Thu Jun 12 17:13:23 2008 +0800
+++ b/suites/libvirt-cim/cimtest/RASD/02_enum.py	Tue Jun 10 11:04:31 2008 -0700
@@ -34,7 +34,6 @@
 from XenKvmLib.vxml import get_class
 from XenKvmLib.classes import get_typed_class
 from XenKvmLib import rasd 
-from XenKvmLib.const import CIM_REV
 from CimTest.Globals import logger
 from CimTest.ReturnCodes import PASS, FAIL
 from XenKvmLib import rasd
diff -r 6652ca49942c -r e1f221334033 suites/libvirt-cim/cimtest/ReferencedProfile/01_verify_refprof.py
--- a/suites/libvirt-cim/cimtest/ReferencedProfile/01_verify_refprof.py	Thu Jun 12 17:13:23 2008 +0800
+++ b/suites/libvirt-cim/cimtest/ReferencedProfile/01_verify_refprof.py	Tue Jun 10 11:04:31 2008 -0700
@@ -79,12 +79,9 @@
 from CimTest.Globals import do_main
 from XenKvmLib.classes import get_typed_class
 from CimTest.ReturnCodes import FAIL, PASS, SKIP
-from XenKvmLib.const import CIM_REV
 from XenKvmLib.common_util import print_field_error
 
 sup_types = ['Xen', 'KVM', 'XenFV', 'LXC']
-
-libvirtcim_rev = 501
 
 def get_proflist():
     proflist = []
@@ -167,12 +164,6 @@
     server = options.ip
     status = PASS
 
-    # Referenced Profile was introduced as part of changeset 501 
-    # and is not available in the libvirt-cim rpm, hence skipping tc
-    # if CIM_REV  501
-    if CIM_REV < libvirtcim_rev:
-        return SKIP
-
     prev_namespace = Globals.CIM_NS
     Globals.CIM_NS = 'root/interop'
     reg_classname = get_typed_class(virt, 'RegisteredProfile')
diff -r 6652ca49942c -r e1f221334033 suites/libvirt-cim/cimtest/ReferencedProfile/02_refprofile_errs.py
--- a/suites/libvirt-cim/cimtest/ReferencedProfile/02_refprofile_errs.py	Thu Jun 12 17:13:23 2008 +0800
+++ b/suites/libvirt-cim/cimtest/ReferencedProfile/02_refprofile_errs.py	Tue Jun 10 11:04:31 2008 -0700
@@ -60,10 +60,8 @@
 from XenKvmLib.classes import get_typed_class
 from CimTest.ReturnCodes import FAIL, PASS, SKIP
 from XenKvmLib.common_util import try_assoc
-from XenKvmLib.const import CIM_REV
 
 sup_types = ['Xen', 'KVM', 'XenFV', 'LXC']
-libvirtcim_rev = 501
 
 expr_values = {
                 'INVALID_Instid_KeyName'  :  {
@@ -124,11 +122,6 @@
     virt = options.virt
     server = options.ip
     status = PASS
-    # Referenced Profile was introduced as part of changeset 501 
-    # and is not available in the libvirt-cim rpm, hence skipping tc
-    # if CIM_REV  501
-    if CIM_REV < libvirtcim_rev:
-        return SKIP
 
     prev_namespace = Globals.CIM_NS
     Globals.CIM_NS = 'root/interop'
diff -r 6652ca49942c -r e1f221334033 suites/libvirt-cim/cimtest/ResourceAllocationFromPool/02_reverse.py
--- a/suites/libvirt-cim/cimtest/ResourceAllocationFromPool/02_reverse.py	Thu Jun 12 17:13:23 2008 +0800
+++ b/suites/libvirt-cim/cimtest/ResourceAllocationFromPool/02_reverse.py	Tue Jun 10 11:04:31 2008 -0700
@@ -29,7 +29,6 @@
 from XenKvmLib.test_doms import destroy_and_undefine_all
 from XenKvmLib.vxml import get_class
 from XenKvmLib.classes import get_typed_class
-from XenKvmLib.const import CIM_REV
 from CimTest import Globals
 from CimTest.Globals import logger, do_main
 from CimTest.ReturnCodes import PASS, FAIL
@@ -43,7 +42,6 @@
 test_vcpus  = 1
 test_mem    = 128
 test_mac    = "00:11:22:33:44:aa"
-proc_instid_rev = 590
 
 def setup_env(server, virt):
     destroy_and_undefine_all(server)
@@ -73,12 +71,7 @@
 
 def init_list(test_disk, diskid, test_network, virt='Xen'):
 
-    if CIM_REV < proc_instid_rev:
-        procid = '%s/%s' % (test_dom, 0)
-    else:
-        procid = '%s/%s' % (test_dom, 'proc')
-
-    proc = { 'rasd_id' : procid,
+    proc = { 'rasd_id' : '%s/%s' % (test_dom, 'proc'),
              'pool_id' : 'ProcessorPool/0'
            }
 
diff -r 6652ca49942c -r e1f221334033 suites/libvirt-cim/cimtest/ResourceAllocationFromPool/03_forward_errs.py
--- a/suites/libvirt-cim/cimtest/ResourceAllocationFromPool/03_forward_errs.py	Thu Jun 12 17:13:23 2008 +0800
+++ b/suites/libvirt-cim/cimtest/ResourceAllocationFromPool/03_forward_errs.py	Tue Jun 10 11:04:31 2008 -0700
@@ -28,11 +28,9 @@
 from CimTest import Globals
 from CimTest.Globals import logger, do_main
 from CimTest.ReturnCodes import PASS, FAIL, XFAIL
-from XenKvmLib.const import CIM_REV
 
 sup_types = ['Xen', 'XenFV', 'KVM']
 
-rev = 466
 
 @do_main(sup_types)
 def main():
@@ -49,9 +47,6 @@
 
     exp_rc = 6 #CIM_ERR_NOT_FOUND
     exp_desc = "No such instance (wrong) - resource pool type mismatch"
-    if CIM_REV < rev:
-        exp_rc = 1 # CIM_ERR_FAILED
-        exp_desc = "Invalid InstanceID or unsupported pool type"
 
     for k, v in poollist.items():
         instanceref = CIMInstanceName(k, 
diff -r 6652ca49942c -r e1f221334033 suites/libvirt-cim/cimtest/SettingsDefine/01_forward.py
--- a/suites/libvirt-cim/cimtest/SettingsDefine/01_forward.py	Thu Jun 12 17:13:23 2008 +0800
+++ b/suites/libvirt-cim/cimtest/SettingsDefine/01_forward.py	Tue Jun 10 11:04:31 2008 -0700
@@ -32,7 +32,6 @@
 from XenKvmLib import assoc
 from XenKvmLib import devices
 from XenKvmLib.classes import get_typed_class
-from XenKvmLib.const import CIM_REV
 from CimTest import Globals
 from CimTest.Globals import do_main
 from CimTest.ReturnCodes import PASS, FAIL 
@@ -42,8 +41,6 @@
 test_dom = "domu1"
 test_mac = "00:11:22:33:44:aa"
 test_vcpus = 1
-proc_instid_rev = 590
-
 
 def print_error(cn, detail):
     Globals.logger.error(Globals.CIM_ERROR_GETINSTANCE, cn)
@@ -96,7 +93,7 @@
     for cn in cn_id.keys():
         key_list = get_keys(cn, cn_id[cn], 'ComputerSystem', options.virt)
 
-        if CIM_REV >= proc_instid_rev and cn == 'Processor':
+        if cn == 'Processor':
             exp_inst_id_val[cn] = "%s/%s" % (test_dom, "proc") 
         else:
             exp_inst_id_val[cn] = key_list['DeviceID']
diff -r 6652ca49942c -r e1f221334033 suites/libvirt-cim/cimtest/SettingsDefine/02_reverse.py
--- a/suites/libvirt-cim/cimtest/SettingsDefine/02_reverse.py	Thu Jun 12 17:13:23 2008 +0800
+++ b/suites/libvirt-cim/cimtest/SettingsDefine/02_reverse.py	Tue Jun 10 11:04:31 2008 -0700
@@ -59,8 +59,6 @@
 from XenKvmLib import vxml
 from XenKvmLib.classes import get_typed_class, get_class_basename
 from XenKvmLib.rasd import InstId_err
-from XenKvmLib.const import CIM_REV
-
 
 sup_types = ['Xen', 'KVM', 'XenFV', 'LXC']
 
@@ -68,9 +66,6 @@
 test_vcpus  = 1
 test_mem    = 128
 test_mac    = "00:11:22:33:44:aa"
-proc_instid_rev = 590
-
-
 
 def call_assoc(ip, inst, exp_id, ccn, virt):
     if inst['InstanceID'] != exp_id:
@@ -196,9 +191,6 @@
             'DiskResourceAllocationSettingData' : '%s/%s' % (test_dom, test_disk),
             'MemResourceAllocationSettingData'  : '%s/%s' % (test_dom, 'mem')}
 
-    if CIM_REV < proc_instid_rev :
-        rasd_devid['ProcResourceAllocationSettingData'] = "%s/%s" % (test_dom, "0")
-
     global dev_devid
     dev_devid = {
             'Processor'   : '%s/%s' % (test_dom, test_vcpus-1),
diff -r 6652ca49942c -r e1f221334033 suites/libvirt-cim/cimtest/SettingsDefineCapabilities/01_forward.py
--- a/suites/libvirt-cim/cimtest/SettingsDefineCapabilities/01_forward.py	Thu Jun 12 17:13:23 2008 +0800
+++ b/suites/libvirt-cim/cimtest/SettingsDefineCapabilities/01_forward.py	Tue Jun 10 11:04:31 2008 -0700
@@ -65,14 +65,11 @@
 from XenKvmLib.common_util import cleanup_restore, create_diskpool_conf, \
 create_netpool_conf
 from XenKvmLib.common_util import print_field_error
-from XenKvmLib.const import CIM_REV
 
 platform_sup = ['Xen', 'KVM', 'XenFV', 'LXC']
 
 memid = "%s/%s" % ("MemoryPool", 0)
 procid = "%s/%s" % ("ProcessorPool", 0)
-libvirtcim_sdc_rasd_rev = 571
-
 
 def get_or_bail(virt, ip, id, pool_class):
     """
@@ -165,27 +162,6 @@
                               rtype[cllist[loop]])
             return FAIL 
 
-        # The following properties have been removed in the patchset 571
-        # but is present in the rpm libvirt-cim and hence retained it.
-
-        if CIM_REV < libvirtcim_sdc_rasd_rev:
-            ppolicy = inst['PropertyPolicy']
-            if ppolicy != 0 and ppolicy != 1:
-                print_field_error("PropertyPolicy", inst['PropertyPolicy'], 
-                                   ppolicy)
-                return FAIL 
-
-            vrole  = inst['ValueRole']
-            if vrole < 0 or vrole > 4:
-                print_field_error("ValueRole", inst['ValueRole'], vrole)
-                return FAIL 
-
-            insid  = inst['InstanceID']
-            vrange = rangelist[insid]
-            if vrange != inst['ValueRange']:
-                print_field_error("ValueRange", inst['ValueRange'], vrange)
-                return FAIL 
-
     return PASS
 
 def verify_sdc_with_ac(virt, server, dpool, npool, mpool, ppool):
diff -r 6652ca49942c -r e1f221334033 suites/libvirt-cim/cimtest/VSSD/04_vssd_to_rasd.py
--- a/suites/libvirt-cim/cimtest/VSSD/04_vssd_to_rasd.py	Thu Jun 12 17:13:23 2008 +0800
+++ b/suites/libvirt-cim/cimtest/VSSD/04_vssd_to_rasd.py	Tue Jun 10 11:04:31 2008 -0700
@@ -53,7 +53,6 @@
 from XenKvmLib.classes import get_typed_class
 from XenKvmLib.rasd import verify_procrasd_values, verify_netrasd_values, \
 verify_diskrasd_values, verify_memrasd_values 
-from XenKvmLib.const import CIM_REV
 
 sup_types = ['Xen', 'KVM', 'XenFV', 'LXC']
 
@@ -61,8 +60,6 @@
 test_vcpus  = 1
 test_mem    = 128
 test_mac    = "00:11:22:33:44:aa"
-proc_rev = 531
-mem_rev = 529
 
 def setup_env(virt):  
     vsxml_info = None
@@ -97,8 +94,6 @@
                  "ResourceType" : 3,
                  "CreationClassName": get_typed_class(virt, 'ProcResourceAllocationSettingData')
                 }
-    if CIM_REV < proc_rev:
-        procrasd['InstanceID'] = '%s/%s' %(test_dom, "0")
 
     netrasd = {
                 "InstanceID"  : '%s/%s' %(test_dom,test_mac), 
@@ -122,8 +117,7 @@
                "VirtualQuantity" : (test_mem * 1024), 
                "CreationClassName": get_typed_class(virt, 'MemResourceAllocationSettingData')
               }
-    if CIM_REV < mem_rev:
-        memrasd['AllocationUnits'] = "MegaBytes"
+
     return procrasd, netrasd, diskrasd, memrasd
 
 def get_inst_from_list(classname, vssd_list, filter_name, exp_val):
diff -r 6652ca49942c -r e1f221334033 suites/libvirt-cim/cimtest/VirtualSystemManagementService/02_destroysystem.py
--- a/suites/libvirt-cim/cimtest/VirtualSystemManagementService/02_destroysystem.py	Thu Jun 12 17:13:23 2008 +0800
+++ b/suites/libvirt-cim/cimtest/VirtualSystemManagementService/02_destroysystem.py	Tue Jun 10 11:04:31 2008 -0700
@@ -28,14 +28,12 @@
 from VirtLib.live import domain_list, active_domain_list
 from XenKvmLib import vsms, vxml
 from XenKvmLib.classes import get_typed_class
-from XenKvmLib.const import CIM_REV
 from CimTest.Globals import do_main
 from CimTest.Globals import logger
 from CimTest.ReturnCodes import PASS, FAIL
 
 sup_types = ['Xen', 'KVM', 'XenFV', 'LXC']
 default_dom = 'test_domain'
-rev = 528
 
 @do_main(sup_types)
 def main():
@@ -50,11 +48,7 @@
     cs_ref = CIMInstanceName(classname, keybindings = {
                                         'Name':default_dom,
                                         'CreationClassName':classname})
-    if CIM_REV < rev:
-        dl_func = active_domain_list
-    else:
-        dl_func = domain_list
-    list_before = dl_func(options.ip, options.virt)
+    list_before = domain_list(options.ip, options.virt)
     status = PASS
     rc = -1
     
@@ -66,7 +60,7 @@
         logger.error(details)
         status = FAIL
 
-    list_after = dl_func(options.ip, options.virt)
+    list_after = domain_list(options.ip, options.virt)
 
     status = PASS
     if default_dom not in list_before:
diff -r 6652ca49942c -r e1f221334033 suites/libvirt-cim/cimtest/VirtualSystemManagementService/05_destroysystem_neg.py
--- a/suites/libvirt-cim/cimtest/VirtualSystemManagementService/05_destroysystem_neg.py	Thu Jun 12 17:13:23 2008 +0800
+++ b/suites/libvirt-cim/cimtest/VirtualSystemManagementService/05_destroysystem_neg.py	Tue Jun 10 11:04:31 2008 -0700
@@ -27,7 +27,6 @@
 from VirtLib import utils
 from XenKvmLib import vsms
 from XenKvmLib.classes import get_typed_class
-from XenKvmLib.const import CIM_REV
 from XenKvmLib.test_doms import undefine_test_domain
 from CimTest.Globals import logger
 from CimTest.Globals import do_main
@@ -45,25 +44,19 @@
         cs_ref = CIMInstanceName(classname, 
                               keybindings = {'CreationClassName':classname})
 
-        if CIM_REV < vsms_status_version:
-            exp_rc = 2 #IM_RC_FAILED
-        else:
-            exp_value = { 'rc'    : pywbem.CIM_ERR_FAILED,
-                          'desc'  : 'CIM_ERR_FAILED: Unable to retrieve domain\
+        exp_value = { 'rc'    : pywbem.CIM_ERR_FAILED,
+                      'desc'  : 'CIM_ERR_FAILED: Unable to retrieve domain\
  name.'
-                        }
+                    }
 
     elif tc == 'nonexistent':
         cs_ref = CIMInstanceName(classname,keybindings = {
                                 'Name':'##@@!!cimtest_domain',
                                 'CreationClassName':classname})
 
-        if CIM_REV < vsms_status_version:
-            exp_rc = 4 #IM_RC_SYS_NOT_FOUND
-        else:
-            exp_value = { 'rc'   : pywbem.CIM_ERR_FAILED,
-                          'desc' : 'CIM_ERR_FAILED: Failed to find domain' 
-                        }
+        exp_value = { 'rc'   : pywbem.CIM_ERR_FAILED,
+                      'desc' : 'CIM_ERR_FAILED: Failed to find domain' 
+                    }
 
     else:
         return SKIP
@@ -71,25 +64,15 @@
     status = FAIL
     try:
         ret = service.DestroySystem(AffectedSystem=cs_ref)
-        if CIM_REV < vsms_status_version:
-            if ret[0] == exp_rc:
-                logger.info('destroy_fail>>%s: Got expected return code %s', 
-                            tc, exp_rc)
-                return PASS 
-            else:
-                logger.error('destroy_fail>>%s: Got rc: %s, but we expect %s',
-                            tc, ret[0], exp_rc)
-                return FAIL 
 
     except Exception, details:
-        if CIM_REV >= vsms_status_version:
-            err_no   = details[0]
-            err_desc = details[1]
-            if err_no == exp_value['rc'] and err_desc == exp_value['desc']:
-                logger.error("For Invalid Scenario '%s'", tc)
-                logger.info('Got expected error no: %s', err_no)
-                logger.info('Got expected error desc: %s',err_desc)
-                return PASS
+        err_no   = details[0]
+        err_desc = details[1]
+        if err_no == exp_value['rc'] and err_desc == exp_value['desc']:
+            logger.error("For Invalid Scenario '%s'", tc)
+            logger.info('Got expected error no: %s', err_no)
+            logger.info('Got expected error desc: %s',err_desc)
+            return PASS
 
         logger.error('destroy_fail>> %s: Error executing DestroySystem' % tc)
         logger.error(details)
diff -r 6652ca49942c -r e1f221334033 suites/libvirt-cim/cimtest/VirtualSystemSettingDataComponent/02_reverse.py
--- a/suites/libvirt-cim/cimtest/VirtualSystemSettingDataComponent/02_reverse.py	Thu Jun 12 17:13:23 2008 +0800
+++ b/suites/libvirt-cim/cimtest/VirtualSystemSettingDataComponent/02_reverse.py	Tue Jun 10 11:04:31 2008 -0700
@@ -55,7 +55,6 @@
 from XenKvmLib.rasd import InstId_err
 from CimTest.Globals import logger, do_main
 from CimTest.ReturnCodes import PASS, FAIL
-from XenKvmLib.const import CIM_REV
 
 sup_types = ['Xen', 'XenFV', 'KVM', 'LXC']
 
@@ -63,7 +62,6 @@
 test_vcpus  = 1
 test_mem    = 128
 test_mac    = "00:11:22:33:44:aa"
-rev = 531
 
 def check_rasd_values(id, exp_id):
     try:
@@ -90,8 +88,6 @@
                  "disk_rasd" : '%s/%s' %(test_dom, test_disk),
                  "mem_rasd"  : '%s/%s' %(test_dom, "mem")
                 }
-    if CIM_REV < rev:
-        rasd_list['proc_rasd'] = '%s/%s' %(test_dom, "0")
 
     try: 
         if len(assoc_info) <= 0: 
diff -r 6652ca49942c -r e1f221334033 suites/libvirt-cim/cimtest/VirtualSystemSettingDataComponent/03_vssdc_fwd_errs.py
--- a/suites/libvirt-cim/cimtest/VirtualSystemSettingDataComponent/03_vssdc_fwd_errs.py	Thu Jun 12 17:13:23 2008 +0800
+++ b/suites/libvirt-cim/cimtest/VirtualSystemSettingDataComponent/03_vssdc_fwd_errs.py	Tue Jun 10 11:04:31 2008 -0700
@@ -62,14 +62,12 @@
 from CimTest.ReturnCodes import PASS, FAIL
 from CimTest.Globals import do_main, logger
 from CimTest.Globals import CIM_USER, CIM_PASS, CIM_NS
-from XenKvmLib.const import CIM_REV
 
 sup_types = ['Xen', 'XenFV', 'KVM', 'LXC']
 
 test_dom     = "domu1"
 test_mac     = "00:11:22:33:44:aa"
 test_vcpus   = 1
-rev = 417
 
 expr_values = {
     "INVALID_InstID_Keyname"   : { 'rc'   : pywbem.CIM_ERR_FAILED, \
@@ -77,10 +75,6 @@
     "INVALID_InstID_Keyval"    : { 'rc'   : pywbem.CIM_ERR_NOT_FOUND, \
                      'desc' : 'No such instance (INVALID_InstID_Keyval)'}
 }
-
-if CIM_REV < rev:
-    expr_values['INVALID_InstID_Keyval'] = {'rc' : pywbem.CIM_ERR_FAILED, \
-                                            'desc' : 'Invalid InstanceID'}
 
 def try_invalid_assoc(classname, name_val, i, field, virt="Xen"):
     ac_classname = get_typed_class(virt, "VirtualSystemSettingDataComponent")
diff -r 6652ca49942c -r e1f221334033 suites/libvirt-cim/lib/XenKvmLib/common_util.py
--- a/suites/libvirt-cim/lib/XenKvmLib/common_util.py	Thu Jun 12 17:13:23 2008 +0800
+++ b/suites/libvirt-cim/lib/XenKvmLib/common_util.py	Tue Jun 10 11:04:31 2008 -0700
@@ -33,17 +33,11 @@
 from XenKvmLib.classes import get_typed_class
 from CimTest.Globals import logger, log_param, CIM_ERROR_ENUMERATE
 from CimTest.ReturnCodes import PASS, FAIL, XFAIL_RC
-from XenKvmLib.const import CIM_REV
 from VirtLib.live import diskpool_list, virsh_version, net_list
 from XenKvmLib.vxml import PoolXML, NetXML
 
 test_dpath = "foo"
-diskpoolconf_rev = 558
-
-if CIM_REV < diskpoolconf_rev:
-    disk_file = '/tmp/diskpool.conf'
-else:
-    disk_file = '/etc/libvirt/diskpool.conf'
+disk_file = '/etc/libvirt/diskpool.conf'
 
 back_disk_file = disk_file + "." + "backup"
 
@@ -276,11 +270,10 @@
     status = PASS
     libvirt_version = virsh_version(server, virt)
     # The conf file is not present on  the machine if 
-    # libvirt_version >= 0.4.1 and CIM_REV > 558
+    # libvirt_version >= 0.4.1
     # Hence Skipping the logic to delete the new conf file
     # and just returning PASS
-    if libvirt_version >= '0.4.1' and \
-       CIM_REV > diskpoolconf_rev:
+    if libvirt_version >= '0.4.1':
         return status
     try:
         if os.path.exists(back_disk_file):
@@ -328,8 +321,7 @@
 
 def create_diskpool_conf(server, virt):
     libvirt_version = virsh_version(server, virt)
-    if libvirt_version >= '0.4.1' and \
-       CIM_REV > diskpoolconf_rev:
+    if libvirt_version >= '0.4.1':
         status, dpoolname = create_diskpool(server, virt=virt)
         diskid = "%s/%s" % ("DiskPool", dpoolname)
     else:
diff -r 6652ca49942c -r e1f221334033 suites/libvirt-cim/lib/XenKvmLib/rasd.py
--- a/suites/libvirt-cim/lib/XenKvmLib/rasd.py	Thu Jun 12 17:13:23 2008 +0800
+++ b/suites/libvirt-cim/lib/XenKvmLib/rasd.py	Tue Jun 10 11:04:31 2008 -0700
@@ -23,7 +23,6 @@
 import sys
 from CimTest.Globals import log_param, logger
 from CimTest.ReturnCodes import FAIL, PASS
-from XenKvmLib.const import CIM_REV
 from XenKvmLib import vxml
 from XenKvmLib.classes import get_typed_class
 
@@ -36,10 +35,6 @@
 memcn  =  'Memory'
 netcn  =  'NetworkPort'
 diskcn =  'LogicalDisk'
-
-
-mem_units_rev = 529
-proc_instid_rev = 590
 
 def rasd_init_list(vsxml, virt, t_disk, t_dom, t_mac, t_mem):
     """
@@ -60,20 +55,9 @@
 
         disk_path = vsxml.xml_get_disk_source()
 
-        if CIM_REV < mem_units_rev:
-          alloc_units  = "MegaBytes"
-        else:
-          alloc_units  = "KiloBytes"
-
-        if CIM_REV < proc_instid_rev:
-           proc_id = '%s/%s' %(t_dom, 0)
-        else:
-           proc_id = '%s/%s' %(t_dom, "proc")
-       
-
         rasd_values = { 
                         proc_cn  : {
-                                     "InstanceID"   : proc_id,
+                                     "InstanceID"   : '%s/%s' %(t_dom, "proc"),
                                      "ResourceType" : 3,
                                     }, 
                         disk_cn  : {
@@ -90,7 +74,7 @@
                         mem_cn   : {
                                     "InstanceID" : '%s/%s' %(t_dom, "mem"), 
                                     "ResourceType"    : 4, 
-                                    "AllocationUnits" : alloc_units,
+                                    "AllocationUnits" : "KiloBytes",
                                     "VirtualQuantity" : (t_mem * 1024),
                                   }
                       } 




More information about the Libvirt-cim mailing list