Alona Kaplan has submitted this change and it was merged. Change subject: core: MacPool per Cluster; update db so that cluster references pool ...................................................................... core: MacPool per Cluster; update db so that cluster references pool • added new column to cluster table • migrated data from StoragePool table — since each cluster has exactly one datacenter, it's straighforward to move pool reference from DC to cluster. • updated adding / modifying / querying of cluster so that mac_pool_id is added / modified / returned respectively. • added query to get pool by cluster id. Bug-Url: https://bugzilla.redhat.com/994283 Change-Id: Id01a33276e3227d7e8bb4b1c8eabbc3af1cafe07 Signed-off-by: Martin Mucha <mmucha(a)redhat.com&gt; --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/Cluster.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/ClusterDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/ClusterDaoImpl.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/network/VmNicDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/network/VmNicDaoImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/ClusterDaoTest.java M backend/manager/modules/dal/src/test/resources/fixtures.xml M packaging/dbscripts/clusters_sp.sql M packaging/dbscripts/mac_pools_sp.sql M packaging/dbscripts/network_sp.sql A packaging/dbscripts/upgrade/04_00_0720_add_mac_pool_id_to_vds_group.sql 11 files changed, 134 insertions(+), 16 deletions(-) Approvals: Eli Mesika: Looks good to me, but someone else must approve Martin Mucha: Verified Alona Kaplan: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/50755 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id01a33276e3227d7e8bb4b1c8eabbc3af1cafe07 Gerrit-PatchSet: 15 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com&gt; Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com&gt; Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com&gt; Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com&gt; Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com&gt; Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com&gt; Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org&gt;