Change in ovirt-engine[master]: core: use AutoStartVmsRunner job from VdsUpdateRunTimeInfo

ofrenkel at redhat.com ofrenkel at redhat.com
Mon Dec 23 09:51:21 UTC 2013


Omer Frenkel has submitted this change and it was merged.

Change subject: core: use AutoStartVmsRunner job from VdsUpdateRunTimeInfo
......................................................................


core: use AutoStartVmsRunner job from VdsUpdateRunTimeInfo

Currently VdsUpdateRunTimeInfo invokes RunVmCommand for each HA VM that
went down and only when it fails because of lock acquisition problem,
the VM is added to the AutoStartVmsRunner job.

This patch changes that, such that the VdsUpdateRunTimeInfo pass the VM
to the AutoStartVmsRunner right away and it will try to run the VM in
its next iteration.

Change-Id: Ia5ff535810f969255fb1d496a11378c2c10bcb91
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsEventListener.java
2 files changed, 2 insertions(+), 22 deletions(-)

Approvals:
  Omer Frenkel: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia5ff535810f969255fb1d496a11378c2c10bcb91
Gerrit-PatchSet: 14
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list