Change in ovirt-engine[master]: core: VM pinning to multiple hosts (database refactor)

rgolan at redhat.com rgolan at redhat.com
Thu Jul 16 10:53:27 UTC 2015


Roy Golan has submitted this change and it was merged.

Change subject: core: VM pinning to multiple hosts (database refactor)
......................................................................


core: VM pinning to multiple hosts (database refactor)

1. Create connection table vm_vds_pinning_map.
   Many to many relations from vm_static to vds_static
2. Populate vm_vds_pinning_map with existing dedicated_vm_for_vds
   fields.
3. Create funtion getDedicatedHostsIdsByVmId(v_vm_id UUID) in
   create_functions.sql
   Returns csv list of dedicated hosts guids
4. Create a view on vm_vds_pinning_map (create_views.sql)
   Providing the coresponding host name and vm name.
5. Update dependant views (create_views.sql and create_dwh_views.sql)
6. Refactor CRUD operations for vm_static (vms_sp.sql and
   vm_templates_sp.sql)
7. Refacotr dedicated_vm_for_vds dependant operation (vds_sp.sql)

Bug-Url: https://bugzilla.redhat.com/1107512
Change-Id: I9d6750936553a3f88a8a24291ecb9725611b8618
Signed-off-by: Dudi Maroshi <dudi at redhat.com>
---
M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmStaticDaoTest.java
M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmTemplateDaoTest.java
M backend/manager/modules/dal/src/test/resources/fixtures.xml
M packaging/dbscripts/create_dwh_views.sql
M packaging/dbscripts/create_functions.sql
M packaging/dbscripts/create_views.sql
A packaging/dbscripts/upgrade/03_06_1750_pinning_to_multiple_hosts_connection_table.sql
M packaging/dbscripts/vds_sp.sql
M packaging/dbscripts/vm_templates_sp.sql
M packaging/dbscripts/vms_sp.sql
10 files changed, 279 insertions(+), 168 deletions(-)

Approvals:
  Dudi Maroshi: Verified
  Roy Golan: Verified; Looks good to me, approved; Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/43387
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I9d6750936553a3f88a8a24291ecb9725611b8618
Gerrit-PatchSet: 13
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Dudi Maroshi <dudi at redhat.com>
Gerrit-Reviewer: Dudi Maroshi <dudi at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Sim Zacks <szacks at redhat.com>
Gerrit-Reviewer: Yaniv Dary <ydary at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list