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(a)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(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server