Change in ovirt-engine[master]: core:vm migration in case of failure ...(#818289)

emesika at redhat.com emesika at redhat.com
Sun Jul 15 07:11:25 UTC 2012


Eli Mesika has submitted this change and it was merged.

Change subject: core:vm migration in case of failure ...(#818289)
......................................................................


core:vm migration in case of failure ...(#818289)

vm migration in case of failure is not logged in task manager

https://bugzilla.redhat.com/show_bug.cgi?id=818289

When we have Host A and Host B where Host A is SPM and we have HA VM
running on Host A, if Host A is going to non-operational (can not access
the storage for example) the HA VM will be migrated to Host B and Host B
will become the SPM but we will not see the Migration under the
SetNonOperational job.
The reason is that the SetNonOperational ExceutionContext is not passed
to MigrateAllVms method in order to use this context as the parent job
context and to associate all migration steps with the SetNonOperational
job.
This patch changes the MigrateAllVms call to pass the parent
SetNonOperational job ExceutionContext, this context is used as the
parent ExceutionContext when migration steps are created for each VM.

Change-Id: I437893ccf373af863810696d14ce8fd1ffb0253b
Signed-off-by: Eli Mesika <emesika at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MaintananceVdsCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/SetNonOperationalVdsCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsEventListener.java
3 files changed, 12 insertions(+), 12 deletions(-)

Approvals:
  Eli Mesika: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I437893ccf373af863810696d14ce8fd1ffb0253b
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list