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