[PATCH 0 of 6] bundles of update processor id to relect recent provider changes

# HG changeset patch # User Guolian Yun <yunguol@cn.ibm.com> # Date 1207878595 25200 # Node ID 16ad6528cd7d7ef367d354c1cadd0d258fa09e0a # Parent 19ff9c851ed8cb76b74d158a4c439dfa5f4ccb50 [TEST] update processor id in 02_hostsystem_to_rasd.py to reflect recent provider changes Signed-off-by: Guolian Yun <yunguol@cn.ibm.com> diff -r 19ff9c851ed8 -r 16ad6528cd7d suites/libvirt-cim/cimtest/HostSystem/02_hostsystem_to_rasd.py --- a/suites/libvirt-cim/cimtest/HostSystem/02_hostsystem_to_rasd.py Wed Apr 09 18:00:14 2008 +0530 +++ b/suites/libvirt-cim/cimtest/HostSystem/02_hostsystem_to_rasd.py Thu Apr 10 18:49:55 2008 -0700 @@ -79,7 +79,7 @@ def init_list(vsxml, virt="Xen"): rasd_values = { proc_cn : { - "InstanceID" : '%s/%s' %(test_dom,0), + "InstanceID" : '%s/%s' %(test_dom, "proc"), "ResourceType" : 3, }, disk_cn : {

+1 for me. Guo Lian Yun wrote:
# HG changeset patch # User Guolian Yun <yunguol@cn.ibm.com> # Date 1207878595 25200 # Node ID 16ad6528cd7d7ef367d354c1cadd0d258fa09e0a # Parent 19ff9c851ed8cb76b74d158a4c439dfa5f4ccb50 [TEST] update processor id in 02_hostsystem_to_rasd.py to reflect recent provider changes
Signed-off-by: Guolian Yun <yunguol@cn.ibm.com>
diff -r 19ff9c851ed8 -r 16ad6528cd7d suites/libvirt-cim/cimtest/HostSystem/02_hostsystem_to_rasd.py --- a/suites/libvirt-cim/cimtest/HostSystem/02_hostsystem_to_rasd.py Wed Apr 09 18:00:14 2008 +0530 +++ b/suites/libvirt-cim/cimtest/HostSystem/02_hostsystem_to_rasd.py Thu Apr 10 18:49:55 2008 -0700 @@ -79,7 +79,7 @@ def init_list(vsxml, virt="Xen"):
rasd_values = { proc_cn : { - "InstanceID" : '%s/%s' %(test_dom,0), + "InstanceID" : '%s/%s' %(test_dom, "proc"), "ResourceType" : 3, }, disk_cn : {
_______________________________________________ Libvirt-cim mailing list Libvirt-cim@redhat.com https://www.redhat.com/mailman/listinfo/libvirt-cim

# HG changeset patch # User Guolian Yun <yunguol@cn.ibm.com> # Date 1207878768 25200 # Node ID 1a230184a44b133bb453d9f4582a0ff6e9ea5696 # Parent 16ad6528cd7d7ef367d354c1cadd0d258fa09e0a [TEST] update processor id in Processor01~03 to reflect recent provider changes Signed-off-by: Guolian Yun <yunguol@cn.ibm.com> diff -r 16ad6528cd7d -r 1a230184a44b suites/libvirt-cim/cimtest/Processor/01_processor.py --- a/suites/libvirt-cim/cimtest/Processor/01_processor.py Thu Apr 10 18:49:55 2008 -0700 +++ b/suites/libvirt-cim/cimtest/Processor/01_processor.py Thu Apr 10 18:52:48 2008 -0700 @@ -54,7 +54,7 @@ def main(): logger.error("Domain not started, we're not able to check vcpu") else: for i in range(0, test_vcpus): - devid = "%s/%s" % (test_dom, i) + devid = "%s/%s" % (test_dom, "proc") key_list = { 'DeviceID' : devid, 'CreationClassName' : get_typed_class(options.virt, "Processor"), 'SystemName' : test_dom, diff -r 16ad6528cd7d -r 1a230184a44b suites/libvirt-cim/cimtest/Processor/02_definesys_get_procs.py --- a/suites/libvirt-cim/cimtest/Processor/02_definesys_get_procs.py Thu Apr 10 18:49:55 2008 -0700 +++ b/suites/libvirt-cim/cimtest/Processor/02_definesys_get_procs.py Thu Apr 10 18:52:48 2008 -0700 @@ -57,7 +57,7 @@ def check_processors(procs): procs['SystemName'], default_dom) return 1 - devid = "%s/%s" % (default_dom, test_vcpus - 1) + devid = "%s/%s" % (default_dom, "proc") if proc['DeviceID'] != devid: logger.error("DeviceID %s does not match expected %s.", diff -r 16ad6528cd7d -r 1a230184a44b suites/libvirt-cim/cimtest/Processor/03_proc_gi_errs.py --- a/suites/libvirt-cim/cimtest/Processor/03_proc_gi_errs.py Thu Apr 10 18:49:55 2008 -0700 +++ b/suites/libvirt-cim/cimtest/Processor/03_proc_gi_errs.py Thu Apr 10 18:52:48 2008 -0700 @@ -174,7 +174,7 @@ def main(): options = main.options log_param() - devid = "%s/%s" % (test_dom, "0") + devid = "%s/%s" % (test_dom, "proc") status = PASS # Getting the VS list and deleting the test_dom if it already exists.

Guo Lian Yun wrote:
# HG changeset patch # User Guolian Yun <yunguol@cn.ibm.com> # Date 1207878768 25200 # Node ID 1a230184a44b133bb453d9f4582a0ff6e9ea5696 # Parent 16ad6528cd7d7ef367d354c1cadd0d258fa09e0a [TEST] update processor id in Processor01~03 to reflect recent provider changes
Signed-off-by: Guolian Yun <yunguol@cn.ibm.com>
diff -r 16ad6528cd7d -r 1a230184a44b suites/libvirt-cim/cimtest/Processor/01_processor.py --- a/suites/libvirt-cim/cimtest/Processor/01_processor.py Thu Apr 10 18:49:55 2008 -0700 +++ b/suites/libvirt-cim/cimtest/Processor/01_processor.py Thu Apr 10 18:52:48 2008 -0700 @@ -54,7 +54,7 @@ def main(): logger.error("Domain not started, we're not able to check vcpu") else: for i in range(0, test_vcpus):
The for loop is not required anymore, since we get only one record irrespective of the vcpus assigned to the domain.
- devid = "%s/%s" % (test_dom, i) + devid = "%s/%s" % (test_dom, "proc") key_list = { 'DeviceID' : devid, 'CreationClassName' : get_typed_class(options.virt, "Processor"), 'SystemName' : test_dom
As of now, the test case just gives a log msg, it can be improved by verifying the DeviceID returned as below: if dev.DeviceID != devid: logger.info("Mismatching device, returned %s instead %s" % (dev.DeviceID, devid)) break
diff -r 16ad6528cd7d -r 1a230184a44b suites/libvirt-cim/cimtest/Processor/02_definesys_get_procs.py --- a/suites/libvirt-cim/cimtest/Processor/02_definesys_get_procs.py Thu Apr 10 18:49:55 2008 -0700 +++ b/suites/libvirt-cim/cimtest/Processor/02_definesys_get_procs.py Thu Apr 10 18:52:48 2008 -0700 @@ -57,7 +57,7 @@ def check_processors(procs): procs['SystemName'], default_dom) return 1
- devid = "%s/%s" % (default_dom, test_vcpus - 1) + devid = "%s/%s" % (default_dom, "proc")
looks good. The tc fails for XenFV with the following error: CIM_ERR_FAILED: A general error occurred that is not covered by a more specific error code: "Failed to create domain" ERROR - Unexpected rc code 1 and description: ERROR - Exception: Unable create domain test_domain using DefineSystem() May be this is an issue on my machine only. Also, for setting the status values can we use PASS, FAIL ? As of now the above two tc used 0, 1 for this.
if proc['DeviceID'] != devid: logger.error("DeviceID %s does not match expected %s.", diff -r 16ad6528cd7d -r 1a230184a44b suites/libvirt-cim/cimtest/Processor/03_proc_gi_errs.py --- a/suites/libvirt-cim/cimtest/Processor/03_proc_gi_errs.py Thu Apr 10 18:49:55 2008 -0700 +++ b/suites/libvirt-cim/cimtest/Processor/03_proc_gi_errs.py Thu Apr 10 18:52:48 2008 -0700 @@ -174,7 +174,7 @@ def main(): options = main.options log_param()
- devid = "%s/%s" % (test_dom, "0") + devid = "%s/%s" % (test_dom, "proc")
Looks good.
status = PASS
# Getting the VS list and deleting the test_dom if it already exists.
_______________________________________________ Libvirt-cim mailing list Libvirt-cim@redhat.com https://www.redhat.com/mailman/listinfo/libvirt-cim

logger.error("Domain not started, we're not able to check vcpu") else: for i in range(0, test_vcpus):
Can you remove the loop here? We're only testing one processor, no need for a loop. Also,
- devid = "%s/%s" % (test_dom, i) + devid = "%s/%s" % (test_dom, "proc") key_list = { 'DeviceID' : devid, 'CreationClassName' : get_typed_class(options.virt, "Processor"), 'SystemName' : test_dom, diff -r 16ad6528cd7d -r 1a230184a44b suites/libvirt-cim/cimtest/Processor/02_definesys_get_procs.py --- a/suites/libvirt-cim/cimtest/Processor/02_definesys_get_procs.py Thu Apr 10 18:49:55 2008 -0700 +++ b/suites/libvirt-cim/cimtest/Processor/02_definesys_get_procs.py Thu Apr 10 18:52:48 2008 -0700 @@ -57,7 +57,7 @@ def check_processors(procs): procs['SystemName'], default_dom) return 1
- devid = "%s/%s" % (default_dom, test_vcpus - 1) + devid = "%s/%s" % (default_dom, "proc")
if proc['DeviceID'] != devid: logger.error("DeviceID %s does not match expected %s.", diff -r 16ad6528cd7d -r 1a230184a44b suites/libvirt-cim/cimtest/Processor/03_proc_gi_errs.py --- a/suites/libvirt-cim/cimtest/Processor/03_proc_gi_errs.py Thu Apr 10 18:49:55 2008 -0700 +++ b/suites/libvirt-cim/cimtest/Processor/03_proc_gi_errs.py Thu Apr 10 18:52:48 2008 -0700 @@ -174,7 +174,7 @@ def main(): options = main.options log_param()
- devid = "%s/%s" % (test_dom, "0") + devid = "%s/%s" % (test_dom, "proc") status = PASS
-- Kaitlin Rupert IBM Linux Technology Center kaitlin@linux.vnet.ibm.com

# HG changeset patch # User Guolian Yun <yunguol@cn.ibm.com> # Date 1207878950 25200 # Node ID 682ad781e5b01be0baf81a5db2da840f1986655d # Parent 1a230184a44b133bb453d9f4582a0ff6e9ea5696 [TEST] update processor id in RASD01~02 to reflect recent provider changes Signed-off-by: Guolian Yun <yunguol@cn.ibm.com> diff -r 1a230184a44b -r 682ad781e5b0 suites/libvirt-cim/cimtest/RASD/01_verify_rasd_fields.py --- a/suites/libvirt-cim/cimtest/RASD/01_verify_rasd_fields.py Thu Apr 10 18:52:48 2008 -0700 +++ b/suites/libvirt-cim/cimtest/RASD/01_verify_rasd_fields.py Thu Apr 10 18:55:50 2008 -0700 @@ -74,7 +74,7 @@ def init_list(xml, disk, virt="Xen"): Creating the lists that will be used for comparisons. """ procrasd = { - "InstanceID" : '%s/%s' %(test_dom,0),\ + "InstanceID" : '%s/%s' %(test_dom, "proc"),\ "ResourceType" : 3,\ "CreationClassName": get_typed_class(virt, 'ProcResourceAllocationSettingData') } diff -r 1a230184a44b -r 682ad781e5b0 suites/libvirt-cim/cimtest/RASD/02_enum.py --- a/suites/libvirt-cim/cimtest/RASD/02_enum.py Thu Apr 10 18:52:48 2008 -0700 +++ b/suites/libvirt-cim/cimtest/RASD/02_enum.py Thu Apr 10 18:55:50 2008 -0700 @@ -51,7 +51,7 @@ def init_list(virt="Xen"): Creating the lists that will be used for comparisons. """ procrasd = { - "InstanceID" : '%s/%s' %(test_dom,0),\ + "InstanceID" : '%s/%s' %(test_dom, "proc"),\ "ResourceType" : 3,\ "CreationClassName": get_typed_class(virt, 'ProcResourceAllocationSettingData') }

+1 for me. Guo Lian Yun wrote:
# HG changeset patch # User Guolian Yun <yunguol@cn.ibm.com> # Date 1207878950 25200 # Node ID 682ad781e5b01be0baf81a5db2da840f1986655d # Parent 1a230184a44b133bb453d9f4582a0ff6e9ea5696 [TEST] update processor id in RASD01~02 to reflect recent provider changes
Signed-off-by: Guolian Yun <yunguol@cn.ibm.com>
diff -r 1a230184a44b -r 682ad781e5b0 suites/libvirt-cim/cimtest/RASD/01_verify_rasd_fields.py --- a/suites/libvirt-cim/cimtest/RASD/01_verify_rasd_fields.py Thu Apr 10 18:52:48 2008 -0700 +++ b/suites/libvirt-cim/cimtest/RASD/01_verify_rasd_fields.py Thu Apr 10 18:55:50 2008 -0700 @@ -74,7 +74,7 @@ def init_list(xml, disk, virt="Xen"): Creating the lists that will be used for comparisons. """ procrasd = { - "InstanceID" : '%s/%s' %(test_dom,0),\ + "InstanceID" : '%s/%s' %(test_dom, "proc"),\ "ResourceType" : 3,\ "CreationClassName": get_typed_class(virt, 'ProcResourceAllocationSettingData') } diff -r 1a230184a44b -r 682ad781e5b0 suites/libvirt-cim/cimtest/RASD/02_enum.py --- a/suites/libvirt-cim/cimtest/RASD/02_enum.py Thu Apr 10 18:52:48 2008 -0700 +++ b/suites/libvirt-cim/cimtest/RASD/02_enum.py Thu Apr 10 18:55:50 2008 -0700 @@ -51,7 +51,7 @@ def init_list(virt="Xen"): Creating the lists that will be used for comparisons. """ procrasd = { - "InstanceID" : '%s/%s' %(test_dom,0),\ + "InstanceID" : '%s/%s' %(test_dom, "proc"),\ "ResourceType" : 3,\ "CreationClassName": get_typed_class(virt, 'ProcResourceAllocationSettingData') }
_______________________________________________ Libvirt-cim mailing list Libvirt-cim@redhat.com https://www.redhat.com/mailman/listinfo/libvirt-cim

# HG changeset patch # User Guolian Yun <yunguol@cn.ibm.com> # Date 1207879260 25200 # Node ID 248707005ae8efa00fc106f3ad8da02add06b3b6 # Parent 682ad781e5b01be0baf81a5db2da840f1986655d [TEST] update processor id in SettingsDefine01~03 to reflect recent provider changes Signed-off-by: Guolian Yun <yunguol@cn.ibm.com> diff -r 682ad781e5b0 -r 248707005ae8 suites/libvirt-cim/cimtest/SettingsDefine/01_forward.py --- a/suites/libvirt-cim/cimtest/SettingsDefine/01_forward.py Thu Apr 10 18:55:50 2008 -0700 +++ b/suites/libvirt-cim/cimtest/SettingsDefine/01_forward.py Thu Apr 10 19:01:00 2008 -0700 @@ -82,7 +82,7 @@ def main(): 'LogicalDisk' : test_disk, 'Memory' : 'mem', 'NetworkPort' : test_mac, - 'Processor' : test_vcpus -1 } + 'Processor' : 'proc' } devlist = {} logelelst = {} diff -r 682ad781e5b0 -r 248707005ae8 suites/libvirt-cim/cimtest/SettingsDefine/02_reverse.py --- a/suites/libvirt-cim/cimtest/SettingsDefine/02_reverse.py Thu Apr 10 18:55:50 2008 -0700 +++ b/suites/libvirt-cim/cimtest/SettingsDefine/02_reverse.py Thu Apr 10 19:01:00 2008 -0700 @@ -186,13 +186,13 @@ def main(): global rasd_devid rasd_devid = { - 'ProcResourceAllocationSettingData' : '%s/%s' % (test_dom, test_vcpus-1), + 'ProcResourceAllocationSettingData' : '%s/%s' % (test_dom, 'proc'), 'NetResourceAllocationSettingData' : '%s/%s' % (test_dom, test_mac), 'DiskResourceAllocationSettingData' : '%s/%s' % (test_dom, test_disk), 'MemResourceAllocationSettingData' : '%s/%s' % (test_dom, 'mem')} global dev_devid dev_devid = { - 'Processor' : '%s/%s' % (test_dom, test_vcpus-1), + 'Processor' : '%s/%s' % (test_dom, 'proc'), 'NetworkPort' : '%s/%s' % (test_dom, test_mac), 'LogicalDisk' : '%s/%s' % (test_dom, test_disk), 'Memory' : '%s/%s' % (test_dom, 'mem')} diff -r 682ad781e5b0 -r 248707005ae8 suites/libvirt-cim/cimtest/SettingsDefine/03_sds_fwd_errs.py --- a/suites/libvirt-cim/cimtest/SettingsDefine/03_sds_fwd_errs.py Thu Apr 10 18:55:50 2008 -0700 +++ b/suites/libvirt-cim/cimtest/SettingsDefine/03_sds_fwd_errs.py Thu Apr 10 19:01:00 2008 -0700 @@ -232,7 +232,7 @@ def main(): get_typed_class(options.virt, 'LogicalDisk') : test_disk, get_typed_class(options.virt, 'Memory') : 'mem', get_typed_class(options.virt, 'NetworkPort') : test_mac, - get_typed_class(options.virt, 'Processor') : test_vcpus - 1 + get_typed_class(options.virt, 'Processor') : 'proc' } tc_scen = [

+1 for me. Guo Lian Yun wrote:
# HG changeset patch # User Guolian Yun <yunguol@cn.ibm.com> # Date 1207879260 25200 # Node ID 248707005ae8efa00fc106f3ad8da02add06b3b6 # Parent 682ad781e5b01be0baf81a5db2da840f1986655d [TEST] update processor id in SettingsDefine01~03 to reflect recent provider changes
Signed-off-by: Guolian Yun <yunguol@cn.ibm.com>
diff -r 682ad781e5b0 -r 248707005ae8 suites/libvirt-cim/cimtest/SettingsDefine/01_forward.py --- a/suites/libvirt-cim/cimtest/SettingsDefine/01_forward.py Thu Apr 10 18:55:50 2008 -0700 +++ b/suites/libvirt-cim/cimtest/SettingsDefine/01_forward.py Thu Apr 10 19:01:00 2008 -0700 @@ -82,7 +82,7 @@ def main(): 'LogicalDisk' : test_disk, 'Memory' : 'mem', 'NetworkPort' : test_mac, - 'Processor' : test_vcpus -1 } + 'Processor' : 'proc' }
devlist = {} logelelst = {} diff -r 682ad781e5b0 -r 248707005ae8 suites/libvirt-cim/cimtest/SettingsDefine/02_reverse.py --- a/suites/libvirt-cim/cimtest/SettingsDefine/02_reverse.py Thu Apr 10 18:55:50 2008 -0700 +++ b/suites/libvirt-cim/cimtest/SettingsDefine/02_reverse.py Thu Apr 10 19:01:00 2008 -0700 @@ -186,13 +186,13 @@ def main():
global rasd_devid rasd_devid = { - 'ProcResourceAllocationSettingData' : '%s/%s' % (test_dom, test_vcpus-1), + 'ProcResourceAllocationSettingData' : '%s/%s' % (test_dom, 'proc'), 'NetResourceAllocationSettingData' : '%s/%s' % (test_dom, test_mac), 'DiskResourceAllocationSettingData' : '%s/%s' % (test_dom, test_disk), 'MemResourceAllocationSettingData' : '%s/%s' % (test_dom, 'mem')} global dev_devid dev_devid = { - 'Processor' : '%s/%s' % (test_dom, test_vcpus-1), + 'Processor' : '%s/%s' % (test_dom, 'proc'), 'NetworkPort' : '%s/%s' % (test_dom, test_mac), 'LogicalDisk' : '%s/%s' % (test_dom, test_disk), 'Memory' : '%s/%s' % (test_dom, 'mem')} diff -r 682ad781e5b0 -r 248707005ae8 suites/libvirt-cim/cimtest/SettingsDefine/03_sds_fwd_errs.py --- a/suites/libvirt-cim/cimtest/SettingsDefine/03_sds_fwd_errs.py Thu Apr 10 18:55:50 2008 -0700 +++ b/suites/libvirt-cim/cimtest/SettingsDefine/03_sds_fwd_errs.py Thu Apr 10 19:01:00 2008 -0700 @@ -232,7 +232,7 @@ def main(): get_typed_class(options.virt, 'LogicalDisk') : test_disk, get_typed_class(options.virt, 'Memory') : 'mem', get_typed_class(options.virt, 'NetworkPort') : test_mac, - get_typed_class(options.virt, 'Processor') : test_vcpus - 1 + get_typed_class(options.virt, 'Processor') : 'proc' }
tc_scen = [
_______________________________________________ Libvirt-cim mailing list Libvirt-cim@redhat.com https://www.redhat.com/mailman/listinfo/libvirt-cim

diff -r 682ad781e5b0 -r 248707005ae8 suites/libvirt-cim/cimtest/SettingsDefine/02_reverse.py --- a/suites/libvirt-cim/cimtest/SettingsDefine/02_reverse.py Thu Apr 10 18:55:50 2008 -0700 +++ b/suites/libvirt-cim/cimtest/SettingsDefine/02_reverse.py Thu Apr 10 19:01:00 2008 -0700 @@ -186,13 +186,13 @@ def main():
global rasd_devid rasd_devid = { - 'ProcResourceAllocationSettingData' : '%s/%s' % (test_dom, test_vcpus-1), + 'ProcResourceAllocationSettingData' : '%s/%s' % (test_dom, 'proc'), 'NetResourceAllocationSettingData' : '%s/%s' % (test_dom, test_mac), 'DiskResourceAllocationSettingData' : '%s/%s' % (test_dom, test_disk), 'MemResourceAllocationSettingData' : '%s/%s' % (test_dom, 'mem')}
Daisy, can you resend this patch so that it only has this change? I believe the recent provider changes should have only affected the InstanceID of the ProcRASD class, and not the Processor class InstanceID as well. So, the Processor related changes shouldn't be necessary. -- Kaitlin Rupert IBM Linux Technology Center kaitlin@linux.vnet.ibm.com

libvirt-cim-bounces@redhat.com wrote on 2008-04-15 09:02:03:
diff -r 682ad781e5b0 -r 248707005ae8 suites/libvirt- cim/cimtest/SettingsDefine/02_reverse.py --- a/suites/libvirt-cim/cimtest/SettingsDefine/02_reverse.py Thu Apr 10 18:55:50 2008 -0700 +++ b/suites/libvirt-cim/cimtest/SettingsDefine/02_reverse.py Thu Apr 10 19:01:00 2008 -0700 @@ -186,13 +186,13 @@ def main():
global rasd_devid rasd_devid = { - 'ProcResourceAllocationSettingData' : '%s/%s' % (test_dom, test_vcpus-1), + 'ProcResourceAllocationSettingData' : '%s/%s' % (test_dom, 'proc'), 'NetResourceAllocationSettingData' : '%s/%s' % (test_dom, test_mac), 'DiskResourceAllocationSettingData' : '%s/%s' % (test_dom, test_disk), 'MemResourceAllocationSettingData' : '%s/%s' % (test_dom, 'mem')}
Daisy, can you resend this patch so that it only has this change?
Done it.
I believe the recent provider changes should have only affected the InstanceID of the ProcRASD class, and not the Processor class InstanceID as well.
So, the Processor related changes shouldn't be necessary. -- Kaitlin Rupert IBM Linux Technology Center kaitlin@linux.vnet.ibm.com
_______________________________________________ Libvirt-cim mailing list Libvirt-cim@redhat.com https://www.redhat.com/mailman/listinfo/libvirt-cim

# HG changeset patch # User Guolian Yun <yunguol@cn.ibm.com> # Date 1207879393 25200 # Node ID fa561bc3cae2b4de3cf6f43df61bb647e7f9bdbc # Parent 248707005ae8efa00fc106f3ad8da02add06b3b6 [TEST] update processor id in SystemDevice.01 to reflect recent provider changes Signed-off-by: Guolian Yun <yunguol@cn.ibm.com diff -r 248707005ae8 -r fa561bc3cae2 suites/libvirt-cim/cimtest/SystemDevice/01_forward.py --- a/suites/libvirt-cim/cimtest/SystemDevice/01_forward.py Thu Apr 10 19:01:00 2008 -0700 +++ b/suites/libvirt-cim/cimtest/SystemDevice/01_forward.py Thu Apr 10 19:03:13 2008 -0700 @@ -74,7 +74,7 @@ def main(): get_typed_class(options.virt, "NetworkPort") : '%s/%s' % (test_dom, test_mac), get_typed_class(options.virt, "Memory") : '%s/mem' % test_dom, get_typed_class(options.virt, "LogicalDisk") : '%s/%s' % (test_dom, test_disk), - get_typed_class(options.virt, "Processor") : '%s/%s' % (test_dom, test_cpu-1) + get_typed_class(options.virt, "Processor") : '%s/proc' % test_dom, } key_list = {'DeviceID' : '',

+1 for me. Guo Lian Yun wrote:
# HG changeset patch # User Guolian Yun <yunguol@cn.ibm.com> # Date 1207879393 25200 # Node ID fa561bc3cae2b4de3cf6f43df61bb647e7f9bdbc # Parent 248707005ae8efa00fc106f3ad8da02add06b3b6 [TEST] update processor id in SystemDevice.01 to reflect recent provider changes
Signed-off-by: Guolian Yun <yunguol@cn.ibm.com
diff -r 248707005ae8 -r fa561bc3cae2 suites/libvirt-cim/cimtest/SystemDevice/01_forward.py --- a/suites/libvirt-cim/cimtest/SystemDevice/01_forward.py Thu Apr 10 19:01:00 2008 -0700 +++ b/suites/libvirt-cim/cimtest/SystemDevice/01_forward.py Thu Apr 10 19:03:13 2008 -0700 @@ -74,7 +74,7 @@ def main(): get_typed_class(options.virt, "NetworkPort") : '%s/%s' % (test_dom, test_mac), get_typed_class(options.virt, "Memory") : '%s/mem' % test_dom, get_typed_class(options.virt, "LogicalDisk") : '%s/%s' % (test_dom, test_disk), - get_typed_class(options.virt, "Processor") : '%s/%s' % (test_dom, test_cpu-1) + get_typed_class(options.virt, "Processor") : '%s/proc' % test_dom, }
key_list = {'DeviceID' : '',
_______________________________________________ Libvirt-cim mailing list Libvirt-cim@redhat.com https://www.redhat.com/mailman/listinfo/libvirt-cim

Guo Lian Yun wrote:
# HG changeset patch # User Guolian Yun <yunguol@cn.ibm.com> # Date 1207879393 25200 # Node ID fa561bc3cae2b4de3cf6f43df61bb647e7f9bdbc # Parent 248707005ae8efa00fc106f3ad8da02add06b3b6 [TEST] update processor id in SystemDevice.01 to reflect recent provider changes
Signed-off-by: Guolian Yun <yunguol@cn.ibm.com
diff -r 248707005ae8 -r fa561bc3cae2 suites/libvirt-cim/cimtest/SystemDevice/01_forward.py
I'm going to ignore this patch for now, since I think the Processor class behavior is not supposed to change. Out of this patchset, just resend patch 4 (update processor id in SettingsDefine01~03 to reflect recent provider changes). I've applied the others (except for this one, which I'm going to hold off on for now). Thanks! -- Kaitlin Rupert IBM Linux Technology Center kaitlin@linux.vnet.ibm.com

# HG changeset patch # User Guolian Yun <yunguol@cn.ibm.com> # Date 1207879513 25200 # Node ID cb7da73431b75b11c6e7f01ae9a16003dda71233 # Parent fa561bc3cae2b4de3cf6f43df61bb647e7f9bdbc [TEST] update processor id in VSSD.04 to reflect recent provider changes Signed-off-by: Guolian Yun <yunguol@cn.ibm.com> diff -r fa561bc3cae2 -r cb7da73431b7 suites/libvirt-cim/cimtest/VSSD/04_vssd_to_rasd.py --- a/suites/libvirt-cim/cimtest/VSSD/04_vssd_to_rasd.py Thu Apr 10 19:03:13 2008 -0700 +++ b/suites/libvirt-cim/cimtest/VSSD/04_vssd_to_rasd.py Thu Apr 10 19:05:13 2008 -0700 @@ -82,7 +82,7 @@ def init_list(virt): Creating the lists that will be used for comparisons. """ procrasd = { - "InstanceID" : '%s/%s' %(test_dom,0), + "InstanceID" : '%s/%s' %(test_dom, "proc"), "ResourceType" : 3, "CreationClassName": get_typed_class(virt, 'ProcResourceAllocationSettingData') }

+1 for me Guo Lian Yun wrote:
# HG changeset patch # User Guolian Yun <yunguol@cn.ibm.com> # Date 1207879513 25200 # Node ID cb7da73431b75b11c6e7f01ae9a16003dda71233 # Parent fa561bc3cae2b4de3cf6f43df61bb647e7f9bdbc [TEST] update processor id in VSSD.04 to reflect recent provider changes
Signed-off-by: Guolian Yun <yunguol@cn.ibm.com>
diff -r fa561bc3cae2 -r cb7da73431b7 suites/libvirt-cim/cimtest/VSSD/04_vssd_to_rasd.py --- a/suites/libvirt-cim/cimtest/VSSD/04_vssd_to_rasd.py Thu Apr 10 19:03:13 2008 -0700 +++ b/suites/libvirt-cim/cimtest/VSSD/04_vssd_to_rasd.py Thu Apr 10 19:05:13 2008 -0700 @@ -82,7 +82,7 @@ def init_list(virt): Creating the lists that will be used for comparisons. """ procrasd = { - "InstanceID" : '%s/%s' %(test_dom,0), + "InstanceID" : '%s/%s' %(test_dom, "proc"), "ResourceType" : 3, "CreationClassName": get_typed_class(virt, 'ProcResourceAllocationSettingData') }
_______________________________________________ Libvirt-cim mailing list Libvirt-cim@redhat.com https://www.redhat.com/mailman/listinfo/libvirt-cim
participants (3)
-
Deepti B Kalakeri
-
Guo Lian Yun
-
Kaitlin Rupert