[Engine-patches] Change in ovirt-engine[master]: core: Fix VmDAO.getForImage to return multiple VMs

mkolesni at redhat.com mkolesni at redhat.com
Wed Mar 21 09:37:55 EDT 2012


Mike Kolesnik has submitted this change and it was merged.

Change subject: core: Fix VmDAO.getForImage to return multiple VMs
......................................................................


core: Fix VmDAO.getForImage to return multiple VMs

Since the introduction of the shared disc feature, an image is not
assossiated uniquely to a single VM.

The DAO was fixed to represent this.

Change-Id: I7100a7aa15524d35f664b49248e5a37cbfb6a32c
---
M backend/manager/dbscripts/create_views.sql
M backend/manager/dbscripts/vms_sp.sql
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDAO.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDAODbFacadeImpl.java
M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmDAOTest.java
M backend/manager/modules/dal/src/test/resources/fixtures.xml
6 files changed, 105 insertions(+), 22 deletions(-)

Approvals:
  Mike Kolesnik: Looks good to me, approved
  Allon Mureinik: Verified


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7100a7aa15524d35f664b49248e5a37cbfb6a32c
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Livnat Peer <lpeer at redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni at redhat.com>


More information about the Engine-patches mailing list