Change in ovirt-engine[master]: core: decrease pending memory & cpu from the right vds on ru...

[ahadas at redhat.com]

Arik Hadas has submitted this change and it was merged.

Change subject: core: decrease pending memory & cpu from the right vds on run vm
......................................................................


core: decrease pending memory & cpu from the right vds on run vm

On rerun attempt in RunVm/RunVmOnce flow we could decrease the pending
resources from the wrong VDS, i.e not from the VDS the VM was supposed
to run on, in case dedicated VDS was defined for the VM or a specific
VDS was set in RunVmOnce flow.

This patch changes the selection of the VDS for which we should decrease
the pending resources to use RunVmCommandBase#getCurrentVdsId method
which should always return the VDS on which the VM was about to run on.

Change-Id: I7b82e4ef4f9e5ed60542e756640861f7403be090
Bug-Url: https://bugzilla.redhat.com/1061811
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommandBase.java
1 file changed, 1 insertion(+), 2 deletions(-)

Approvals:
  Omer Frenkel: Looks good to me, approved
  Arik Hadas: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7b82e4ef4f9e5ed60542e756640861f7403be090
Gerrit-PatchSet: 7
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: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server