Change in ovirt-engine[master]: core: Removed force select SPM command params storage pool i...

tnisan at redhat.com tnisan at redhat.com
Wed Jul 17 16:35:01 UTC 2013


Tal Nisan has submitted this change and it was merged.

Change subject: core: Removed force select SPM command params storage pool id
......................................................................


core: Removed force select SPM command params storage pool id

Removed the storage pool id from the force select SPM command parameters
the storage pool id can be found easily since the host is a part of a single
pool anyway (or no pool at all) and we fetch the storage pool from the
database anyway so we can remove this overhead

Change-Id: I649d565b645664bf6b57227cc330494e2e158a8e
Signed-off-by: Tal Nisan <tnisan at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ForceSelectSPMCommand.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/ForceSelectSPMCommandTest.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/ForceSelectSPMParameters.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java
4 files changed, 24 insertions(+), 25 deletions(-)

Approvals:
  Tal Nisan: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I649d565b645664bf6b57227cc330494e2e158a8e
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Alissa Bonas <abonas at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgotliv at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list