Change in ovirt-engine[master]: core: add VMs from the same VDS to auto startup job in bulk

ofrenkel at redhat.com ofrenkel at redhat.com
Mon Jan 6 08:08:18 UTC 2014


Omer Frenkel has submitted this change and it was merged.

Change subject: core: add VMs from the same VDS to auto startup job in bulk
......................................................................


core: add VMs from the same VDS to auto startup job in bulk

VdsUpdateRunTimeInfo checks all the VMs which run on the VDS it
monitors, and at the end of the process it has all the HA VMs that ran
on that VDS and went down. So Instead of iterating over those VMs and
pass each of the separately to the AutoStartVmsRunner job, it now pass
them as a bulk (through the VdsEventListener) which is a bit more
efficient.

Change-Id: I560383c8e73bb1728441fe5786954eba7ba88e60
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AutoStartVmsRunner.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsEventListener.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/IVdsEventListener.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java
4 files changed, 40 insertions(+), 25 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I560383c8e73bb1728441fe5786954eba7ba88e60
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: Eli Mesika <emesika 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