Yaniv Bronhaim 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]> Reviewed-on: http://gerrit.ovirt.org/31411 Reviewed-by: Yaniv Bronhaim <[email protected]> --- M tests/vmTests.py M tests/vmTestsData.py M vdsm/clientIF.py M vdsm/virt/vm.py 4 files changed, 166 insertions(+), 56 deletions(-) Approvals: Yaniv Bronhaim: Looks good to me, approved Dan Kenigsberg: Looks good to me, but someone else must approve Francesco Romani: Verified -- To view, visit http://gerrit.ovirt.org/31411 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic0cc57129e9c8e6545f9a947329adf1f9e82648f Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.5 Gerrit-Owner: Francesco Romani <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
