[Libvirt-cim] [PATCH 0 of 2] Remove devid-style InstanceID support from ELEC and EC.

[Heidi Eckhart]

Kaitlin Rupert wrote:
> Removed devid-style InstanceID support from ELEC - this class now uses just the name of domain.
>
> This caused a needed update in EC - cap_to_cs() no longer needs to parse devid-style InstanceIDs.
>
> _______________________________________________
> Libvirt-cim mailing list
> Libvirt-cim at redhat.com
> https://www.redhat.com/mailman/listinfo/libvirt-cim
>   
There seems to be something broken by the last patch set series (applied 
the two from Dan, followed by the two from Kaitlin).

Before applying patch set:
wbemein http://localhost/root/virt:CIM_EnabledLogicalElementCapabilities

localhost:5988/root/virt:KVM_VirtualSystemManagementCapabilities.InstanceID="localhost.localdomain/0"
localhost:5988/root/virt:KVM_EnabledLogicalElementCapabilities.InstanceID="qemu1/0"

After applying patch set:
wbemein http://localhost/root/virt:CIM_EnabledLogicalElementCapabilities

localhost:5988/root/virt:KVM_VirtualSystemManagementCapabilities.InstanceID="ManagementCapabilities"
localhost:5988/root/virt:Xen_VirtualSystemManagementCapabilities.InstanceID="ManagementCapabilities"
localhost:5988/root/virt:KVM_EnabledLogicalElementCapabilities.InstanceID="qemu1"

The instance of Xen_VirtualSystemManagementCapabilities is wrong. I had 
a look into the code and think this could be easily fixed by starting 
return_vsm_cap() with an connect_by_classname(). Kaitlin, please can you 
incorporate that fix into your patch set ? Thanks.

-- 
Regards

Heidi Eckhart
Software Engineer
Linux Technology Center - Open Hypervisor

heidieck at linux.vnet.ibm.com

**************************************************
IBM Deutschland Entwicklung GmbH
Vorsitzender des Aufsichtsrats: Martin Jetter
Geschaeftsfuehrung: Herbert Kircher
Sitz der Gesellschaft: Boeblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294