Guo Lian Yun wrote:
@@ -67,18 +68,20 @@ def main():
def main():
options = main.options
rc = PASS
+ log_param()
- instanceref = CIMInstanceName("Xen_HostSystem",
+ instanceref = CIMInstanceName(get_typed_class(options.virt, "HostSystem"),
#1
keybindings = {"Name" :
"wrong", "CreationClassName" : "wrong"})
- rc = try_assoc(instanceref, "Xen_HostSystem", exp_rc, exp_desc, options)
+ rc = try_assoc(instanceref, get_typed_class(options.virt, "HostSystem"),
exp_rc, exp_desc, options)
#2
if rc != PASS:
status = FAIL
return status
- instance_cs = CIMInstanceName("Xen_ComputerSystem",
- keybindings = {"Name" : "wrong",
"CreationClassName" : "Xen_ComputerSystem"})
- rc = try_assoc(instance_cs, "Xen_ComputerSystem", exp_rc, exp_desc,
options)
+ instance_cs = CIMInstanceName(get_typed_class(options.virt,
"ComputerSystem"),
#3
+ keybindings = {"Name" :
"wrong",
+ "CreationClassName" :
get_typed_class(options.virt,"ComputerSystem")})
+ rc = try_assoc(instance_cs, get_typed_class(options.virt,
"ComputerSystem"), exp_rc, exp_desc, options)
#4
#1 & #2, #3 & #4 share some common code. I would consider use a variable
to avoid duplicated get_typed_class() call.
if rc != PASS:
status = FAIL
return status
_______________________________________________
Libvirt-cim mailing list
Libvirt-cim(a)redhat.com
https://www.redhat.com/mailman/listinfo/libvirt-cim
--
- Zhengang