Change in ovirt-engine[master]: core: use single thread on update sla policies

ofrenkel at redhat.com ofrenkel at redhat.com
Thu Apr 2 08:14:11 UTC 2015


Omer Frenkel has submitted this change and it was merged.

Change subject: core: use single thread on update sla policies
......................................................................


core: use single thread on update sla policies

when vms monitoring cycle finish, it is calling update sla policies in a
new thread.
the new vms monitoring changed the call to be per-vm instead of one call
for all vms that succeeded to run.

in this patch the previous approach restored, calling once with list of
vms, resulting only one thread for all the vms.

Change-Id: Id7f92bf64d771e351f2aab46874515eb70a713fa
Signed-off-by: Omer Frenkel <ofrenkel at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmsMonitoring.java
1 file changed, 4 insertions(+), 7 deletions(-)

Approvals:
  Gilad Chaplik: Looks good to me, but someone else must approve
  Roy Golan: Looks good to me, approved
  Omer Frenkel: Verified
  Arik Hadas: Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/39424
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Id7f92bf64d771e351f2aab46874515eb70a713fa
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list