Change in ovirt-engine[master]: engine: Update pending counters in two ommited cases

ahadas at redhat.com ahadas at redhat.com
Mon Feb 17 12:40:36 UTC 2014


Arik Hadas has submitted this change and it was merged.

Change subject: engine: Update pending counters in two ommited cases
......................................................................


engine: Update pending counters in two ommited cases

The code missed two code paths that resulted in a VM being scheduled
with no update to the pending counters.

- no or empty cluster policy
- preselected host (scheduling only verified the contraints)

Change-Id: I750dadcca53e402a91bd8b36e68ee730fe177278
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1062996
Signed-off-by: Martin Sivak <msivak at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java
1 file changed, 9 insertions(+), 5 deletions(-)

Approvals:
  Martin Sivák: Looks good to me, but someone else must approve
  Gilad Chaplik: Looks good to me, approved
  Arik Hadas: Verified; Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I750dadcca53e402a91bd8b36e68ee730fe177278
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Sivák <msivak 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: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list