Change in ovirt-engine[master]: core: on handing over a VM mark it as MigratingTo

rgolan at redhat.com rgolan at redhat.com
Wed Apr 3 14:38:17 UTC 2013


Roy Golan has submitted this change and it was merged.

Change subject: core: on handing over a VM mark it as MigratingTo
......................................................................


core: on handing over a VM mark it as MigratingTo

This patch aims to reflect the VM handover more clear so instead of
moving a VM to Unknown it will move to MigratingTo (unless the host is
*already* NonResponging) so tracking a VM status transition wouldn't
puzzle users (at least would be less confusing and more script friendly
as the script won't discover a VM is suddenly Uknown)

Non responding treatment has been changed to collect also VMs that are
migrating to the host (i.e status MigratingTo and run_on_vds == nonRespongindVds)

Change-Id: I0cc072b445ef2616e209fd9dfb30f65abc0ad3da
Signed-off-by: Roy Golan <rgolan at redhat.com>
Bug-Url: https://bugzilla.redhat.com/909932
---
M backend/manager/dbscripts/vms_sp.sql
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsNotRespondingTreatmentCommand.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDAO.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDAODbFacadeImpl.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java
5 files changed, 55 insertions(+), 10 deletions(-)

Approvals:
  Roy Golan: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/13560
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I0cc072b445ef2616e209fd9dfb30f65abc0ad3da
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>



More information about the Engine-commits mailing list