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@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@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