Change in ovirt-engine[master]: core: start 'prestarted VMs' immediately after update

mkublin at redhat.com mkublin at redhat.com
Sun Feb 3 14:37:05 UTC 2013


Michael Kublin has submitted this change and it was merged.

Change subject: core: start 'prestarted VMs' immediately after update
......................................................................


core: start 'prestarted VMs' immediately after update

This patch solves a bug in which after increasing the number of
prestarted VMs of a pool, it takes a while (which is defined by the
configuration value "VmPoolMonitorIntervalInMinutes") to start
more VMs to meet the updated number of prestarted VMs.

The solution is to trigger the pool monitoring job to run immediately,
after UpdateVmPoolWithVmsCommand updates the new properties of the
pool.

Change-Id: I636f496b241c042962cfec10880fbee00ce2bab4
Bug-Url: https://bugzilla.redhat.com/888396
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/dbscripts/vms_sp.sql
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/Backend.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmPoolWithVmsCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmPoolCommandBase.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/interfaces/BackendInternal.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VM.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VMStatus.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/dal/src/test/java/org/ovirt/engine/core/dao/VmDAOTest.java
10 files changed, 89 insertions(+), 16 deletions(-)

Approvals:
  Michael Kublin: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I636f496b241c042962cfec10880fbee00ce2bab4
Gerrit-PatchSet: 8
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: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>



More information about the Engine-commits mailing list