Change in vdsm[ovirt-3.4]: vm: more generic recovery check

ybronhei at redhat.com ybronhei at redhat.com
Thu Aug 14 10:14:41 UTC 2014


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 <fromani at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/31241
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/31421
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
---
M tests/vmTests.py
M tests/vmTestsData.py
M vdsm/clientIF.py
M vdsm/vm.py
4 files changed, 171 insertions(+), 55 deletions(-)

Approvals:
  Yaniv Bronhaim: Looks good to me, approved
  Francesco Romani: Verified



-- 
To view, visit http://gerrit.ovirt.org/31421
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.4
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>


More information about the vdsm-patches mailing list