July 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: scheduling manager
by lhornyak＠redhat.com 16 Jul '13

16 Jul '13

Laszlo Hornyak has submitted this change and it was merged. Change subject: engine: scheduling manager ...................................................................... engine: scheduling manager Inroducing new clsuter policy arch; - Handles cluster policy CRUD. - Handles host selection for a single run/migrate vm iteration. Detailed description: http://wiki.ovirt.org/Features/oVirtSchedulerAPI Change-Id: I782f1556f6d955ab644cb437e786e6ef1fa3ea71 Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/CreateVmVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/MigrateVDSCommand.java 3 files changed, 384 insertions(+), 37 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16111 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I782f1556f6d955ab644cb437e786e6ef1fa3ea71 Gerrit-PatchSet: 17 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: policy units logic implementation
by lhornyak＠redhat.com 16 Jul '13

16 Jul '13

Laszlo Hornyak has submitted this change and it was merged. Change subject: engine: policy units logic implementation ...................................................................... engine: policy units logic implementation Translate VdsSelector logic into policy units: - Migration - MigrationDomain - PinToHost - Memory - CPU - Network Translate VdsLoadBalancing logic into policy units: - None (do nothing) - Even distribution - Power saving Change-Id: Ib29c4257586654392555b6b3675928f187ab96b9 Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/PolicyUnitImpl.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/CPUPolicyUnit.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/EvenDistributionPolicyUnit.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/MemoryPolicyUnit.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/MigrationDomainPolicyUnit.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/MigrationPolicyUnit.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/NetworkPolicyUnit.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/NonePolicyUnit.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/PinToHostPolicyUnit.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/PowerSavingPolicyUnit.java 10 files changed, 902 insertions(+), 0 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16110 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib29c4257586654392555b6b3675928f187ab96b9 Gerrit-PatchSet: 14 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Dao test for cluster policy
by lhornyak＠redhat.com 16 Jul '13

16 Jul '13

Laszlo Hornyak has submitted this change and it was merged. Change subject: engine: Dao test for cluster policy ...................................................................... engine: Dao test for cluster policy Added / Updated dao test for VdsGroupDAO, ClusterPolicyDao PolicyUnitDao Change-Id: I59dd6b97ce5ce75eb307d58e6dac99e35f43381f Signed-off-by: Noam Slomianko <nslomian(a)redhat.com> --- A backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/ClusterPolicyDaoTest.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/FixturesTool.java A backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/PolicyUnitDaoTest.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VdsGroupDAOTest.java M backend/manager/modules/dal/src/test/resources/fixtures.xml 5 files changed, 291 insertions(+), 0 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16452 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I59dd6b97ce5ce75eb307d58e6dac99e35f43381f Gerrit-PatchSet: 14 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Noam Slomianko <nslomian(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Noam Slomianko <nslomian(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: cluster policy db support
by lhornyak＠redhat.com 16 Jul '13

16 Jul '13

Laszlo Hornyak has submitted this change and it was merged. Change subject: engine: cluster policy db support ...................................................................... engine: cluster policy db support Added tables: - cluster policies - policy units - cluster policy units Altered table: - VdsGroup (reference for cluster policy, and policy custom parameters) Change-Id: Ibc34c5eba1d68e0650df38053c97e5138de3dfad Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/DbFacade.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsGroupDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsGroupDAODbFacadeImpl.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/scheduling/ClusterPolicyDao.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/scheduling/ClusterPolicyDaoImpl.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/scheduling/PolicyUnitDao.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/scheduling/PolicyUnitDaoImpl.java M backend/manager/modules/dal/src/main/jdbc-resources/engine-daos.properties A packaging/dbscripts/cluster_policy_sp.sql A packaging/dbscripts/policy_unit_sp.sql A packaging/dbscripts/upgrade/03_03_0480_upgrade_cluster_policy.sql M packaging/dbscripts/vds_groups_sp.sql 12 files changed, 725 insertions(+), 7 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16109 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibc34c5eba1d68e0650df38053c97e5138de3dfad Gerrit-PatchSet: 14 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Noam Slomianko <nslomian(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: common: cluster policy fields in VDSGroup
by lhornyak＠redhat.com 16 Jul '13

16 Jul '13

Laszlo Hornyak has submitted this change and it was merged. Change subject: common: cluster policy fields in VDSGroup ...................................................................... common: cluster policy fields in VDSGroup add to cluster BE the fields: * cluster policy id * cluster policy parameter map Change-Id: Ic22b3730f05d5dde4c8a689613306cf5cbcfa7df Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDSGroup.java 1 file changed, 26 insertions(+), 1 deletion(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16108 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic22b3730f05d5dde4c8a689613306cf5cbcfa7df Gerrit-PatchSet: 12 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: common: Introduce new scheduling arch BEs
by lhornyak＠redhat.com 16 Jul '13

16 Jul '13

Laszlo Hornyak has submitted this change and it was merged. Change subject: common: Introduce new scheduling arch BEs ...................................................................... common: Introduce new scheduling arch BEs * PolicyUnit: data stucture representing a single scheduling logic unit. * ClusterPolicy: container data structure for cluster's policy, contains attached filters, functions and load balancing logic. Detailed design: http://wiki.ovirt.org/Features/oVirtSchedulerAPI Change-Id: I1b6946d51e55b936747c13878d4b710af0267016 Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/scheduling/ClusterPolicy.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/scheduling/PolicyUnit.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml 3 files changed, 381 insertions(+), 0 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16107 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1b6946d51e55b936747c13878d4b710af0267016 Gerrit-PatchSet: 12 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: move static method
by lhornyak＠redhat.com 16 Jul '13

16 Jul '13

Laszlo Hornyak has submitted this change and it was merged. Change subject: engine: move static method ...................................................................... engine: move static method getEffectiveCpuCores from vdsSelector to SchedulingValidator util class Change-Id: I447eec70ecb06daf4e8208f49e4478e4786b1cd0 Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/EvenlyDistributeComparer.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SlaValidator.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/VdsCpuVdsLoadBalancingAlgorithm.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/VdsSelector.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/VmCountVdsLoadBalancingAlgorithm.java 5 files changed, 25 insertions(+), 24 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16106 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I447eec70ecb06daf4e8208f49e4478e4786b1cd0 Gerrit-PatchSet: 12 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-guest-agent[master]: agent: Add possibility to NOT build kdm and/or gdm extension...
by bazulay＠redhat.com 16 Jul '13

16 Jul '13

Barak Azulay has submitted this change and it was merged. Change subject: agent: Add possibility to NOT build kdm and/or gdm extensions ...................................................................... agent: Add possibility to NOT build kdm and/or gdm extensions Since with GNOME 3.8 it's no longer possible to build extensions for GDM we're now allowing to disable the build of the extensions for systems not supporting it. Additionally we're now allowing a better control via configure what to build. Change-Id: I87745ea0e49860969d782c35e2e83cec12fff864 Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M Makefile.am M configure.ac 2 files changed, 85 insertions(+), 54 deletions(-) Approvals: Vinzenz Feenstra: Verified; Looks good to me, but someone else must approve Barak Azulay: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16714 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I87745ea0e49860969d782c35e2e83cec12fff864 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: master Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com>

1 0

Change in ovirt-guest-agent[master]: agent: Adding --with-dist for packaging ease
by bazulay＠redhat.com 16 Jul '13

16 Jul '13

Barak Azulay has submitted this change and it was merged. Change subject: agent: Adding --with-dist for packaging ease ...................................................................... agent: Adding --with-dist for packaging ease Since not all platforms can fullfill the requirements of the configure script, but tar balls need to be creatable anyway, add a --with-dist to configure for being able to create the distribution tarballs anywhere. --with-dist on configure now disables the requirement checks. This is needed as for example F19 or RHEL6/CentOS won't be able to fulfill the requirements for the GDM plugin. Change-Id: If1daa4438d9037d47ed4e7ee0be35cb1c2755779 Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M configure.ac 1 file changed, 14 insertions(+), 3 deletions(-) Approvals: Vinzenz Feenstra: Verified; Looks good to me, but someone else must approve Barak Azulay: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16696 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If1daa4438d9037d47ed4e7ee0be35cb1c2755779 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: master Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com>

1 0

Change in ovirt-guest-agent[master]: agent: RHEL6 spec file
by bazulay＠redhat.com 16 Jul '13

16 Jul '13

Barak Azulay has submitted this change and it was merged. Change subject: agent: RHEL6 spec file ...................................................................... agent: RHEL6 spec file Adding a RHEL6 specific spec file for the ovirt-guest-agent Change-Id: Ife545bcfc597a3c0227ec1b17258234b87b2df7a Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M Makefile.am A ovirt-guest-agent.rhel6.spec 2 files changed, 133 insertions(+), 0 deletions(-) Approvals: Vinzenz Feenstra: Verified; Looks good to me, but someone else must approve Barak Azulay: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16695 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ife545bcfc597a3c0227ec1b17258234b87b2df7a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: master Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Michael Burns <mburns(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com>

1 0