Change in ovirt-engine[master]: core: Add storage types to DiskImage entity

tnisan at redhat.com tnisan at redhat.com
Wed Jan 22 14:06:55 UTC 2014


Tal Nisan has submitted this change and it was merged.

Change subject: core: Add storage types to DiskImage entity
......................................................................


core: Add storage types to DiskImage entity

Currently DiskImage has a storageIds field that contains a list of the
storages it resides on. It makes sense also to add a storage types field
to avoid having to fetch the storage domain in order to know which storage
type the disk resides on.
This will be helpful when the storage pool storage type property will be
removed.

Change-Id: I8302972bb21fa076cd35fb308f5f660ca9200daa
Relates-To: https://bugzilla.redhat.com/1038053
Signed-off-by: Tal Nisan <tnisan at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/DiskImage.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/DiskImageDAODbFacadeImpl.java
M packaging/dbscripts/create_views.sql
3 files changed, 34 insertions(+), 1 deletion(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8302972bb21fa076cd35fb308f5f660ca9200daa
Gerrit-PatchSet: 10
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Liron Ar <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk 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