Dan Kenigsberg has submitted this change and it was merged. Change subject: vm: more generic recovery check ......................................................................
vm: more generic recovery check On recovery, VDSM checks each VM listed by libvirt to see if it should take care of it. The check is done by using some system properties added in the smbios domain element, which is lacking outside x86_64. As result, recovery finds no VMs on PPC64. This patch makes VDSM use the vm guest agent channels to do the recovery check, since they are avaialble on all plataforms. Change-Id: Ic0cc57129e9c8e6545f9a947329adf1f9e82648f Bug-Url: https://bugzilla.redhat.com/1126887 Signed-off-by: Francesco Romani <[email protected]> Reviewed-on: http://gerrit.ovirt.org/31241 Reviewed-by: Dan Kenigsberg <[email protected]> --- M tests/vmTests.py M tests/vmTestsData.py M vdsm/clientIF.py M vdsm/virt/vm.py M vdsm/virt/vmxml.py 5 files changed, 166 insertions(+), 55 deletions(-) Approvals: Dan Kenigsberg: Looks good to me, approved Francesco Romani: Verified -- To view, visit http://gerrit.ovirt.org/31241 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic0cc57129e9c8e6545f9a947329adf1f9e82648f Gerrit-PatchSet: 8 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Francesco Romani <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Eduardo <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
