June 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: scheduling: Add PreferredHosts policy unit
by rgolan＠redhat.com 08 Jun '16

08 Jun '16

Roy Golan has submitted this change and it was merged. Change subject: scheduling: Add PreferredHosts policy unit ...................................................................... scheduling: Add PreferredHosts policy unit This policy unit replaces the old behaviour of destHostIdList to keep the preferred host for VM functionality, but doing it in the "proper" way that can be disabled in the cluster policy. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1243811 Change-Id: I2f55caf42b787458d031296cdd8f837f79293a3b Signed-off-by: Martin Sivák <msivak(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/InternalClusterPolicies.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/InternalPolicyUnits.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/PreferredHostsWeightPolicyUnit.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/policyunits/PreferredHostsWeightPolicyUnitTest.java 6 files changed, 132 insertions(+), 2 deletions(-) Approvals: Martin Sivák: Verified Jenkins CI: Passed CI tests Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/45816 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2f55caf42b787458d031296cdd8f837f79293a3b Gerrit-PatchSet: 36 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Phillip Bailey <phbailey(a)redhat.com> Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Yanir Quinn <yquinn(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.5-gluster]: gluster: fix geo rep session key parsing issue
by sabose＠redhat.com 08 Jun '16

08 Jun '16

Sahina Bose has submitted this change and it was merged. Change subject: gluster: fix geo rep session key parsing issue ...................................................................... gluster: fix geo rep session key parsing issue There is a change in the geo rep ression key from glusterfs 3.7.12 onwards. Slave volume UUID is added as lost component to session key. Updating the session key parsing to handle both old and new versions. Change-Id: I8717817065c17b3d9d50b3ff6f524d7a530f2039 Bug-Url: https://bugzilla.redhat.com/1343650 Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GlusterVolumeGeoRepStatusForXmlRpc.java 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: Sahina Bose: Looks good to me, approved; Passed CI tests Ramesh N: Verified -- To view, visit https://gerrit.ovirt.org/58848 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8717817065c17b3d9d50b3ff6f524d7a530f2039 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: gluster: fix geo rep session key parsing issue
by sabose＠redhat.com 08 Jun '16

08 Jun '16

Sahina Bose has submitted this change and it was merged. Change subject: gluster: fix geo rep session key parsing issue ...................................................................... gluster: fix geo rep session key parsing issue There is a change in the geo rep ression key from glusterfs 3.7.12 onwards. Slave volume UUID is added as lost component to session key. Updating the session key parsing to handle both old and new versions. Change-Id: I8717817065c17b3d9d50b3ff6f524d7a530f2039 Bug-Url: https://bugzilla.redhat.com/1343650 Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GlusterVolumeGeoRepStatusForXmlRpc.java 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: Jenkins CI: Passed CI tests Sahina Bose: Looks good to me, approved Ramesh N: Verified -- To view, visit https://gerrit.ovirt.org/58793 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8717817065c17b3d9d50b3ff6f524d7a530f2039 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Aravinda VK <avishwan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: hide unnecessary label from Edit VM popup
by tnisan＠redhat.com 08 Jun '16

08 Jun '16

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: hide unnecessary label from Edit VM popup ...................................................................... webadmin: hide unnecessary label from Edit VM popup This patch removes the unnecessary label "Disks Allocation:" from the "Resource Allocation" tab in the "Edit Virtual Machine" popup. It sets the visibility of the label to be false so that only when there are disks to display, it is set to true. Change-Id: I422ed365b83d1560c9393910a00570b7e42f7d65 Bug-Url: https://bugzilla.redhat.com/1251928 Signed-off-by: Idan Shaby <ishaby(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/storage/DisksAllocationView.ui.xml 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Idan Shaby: Verified -- To view, visit https://gerrit.ovirt.org/58801 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I422ed365b83d1560c9393910a00570b7e42f7d65 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.0]: node: Allow a separate suffix for the placeholder package
by fabiand＠redhat.com 08 Jun '16

08 Jun '16

Fabian Deutsch has submitted this change and it was merged. Change subject: node: Allow a separate suffix for the placeholder package ...................................................................... node: Allow a separate suffix for the placeholder package This is needed to allow snapshot NGN builds in the stable branches Change-Id: I559b25168fcee4b3c11931f064394da44a3e173d Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M ovirt-release40.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58839 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I559b25168fcee4b3c11931f064394da44a3e173d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.0]: node: Add timestamp to placeholder
by fabiand＠redhat.com 08 Jun '16

08 Jun '16

Fabian Deutsch has submitted this change and it was merged. Change subject: node: Add timestamp to placeholder ...................................................................... node: Add timestamp to placeholder This is needed to allow daily builds of node in the snapshot branch Change-Id: If7c9cca98d9828b4a8366d15cf8631d16204a977 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M automation/build-artifacts.sh 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58840 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If7c9cca98d9828b4a8366d15cf8631d16204a977 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: node: Add hint for an additional bug
by fabiand＠redhat.com 08 Jun '16

08 Jun '16

Fabian Deutsch has submitted this change and it was merged. Change subject: node: Add hint for an additional bug ...................................................................... node: Add hint for an additional bug Change-Id: If26de5b185bcf79d6389749abb3a0f57be5af4d1 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M ovirt-release-master.spec.in 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58837 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If26de5b185bcf79d6389749abb3a0f57be5af4d1 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: node: Drop he-setup NM check
by fabiand＠redhat.com 08 Jun '16

08 Jun '16

Fabian Deutsch has submitted this change and it was merged. Change subject: node: Drop he-setup NM check ...................................................................... node: Drop he-setup NM check This is now in master with https://gerrit.ovirt.org/#/c/56146/ Change-Id: I898aae7ce19b5ca3bb51d2bf531dd7660e6ae97f Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M ovirt-release-master.spec.in 1 file changed, 0 insertions(+), 5 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58836 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I898aae7ce19b5ca3bb51d2bf531dd7660e6ae97f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: node: Drop hack
by fabiand＠redhat.com 08 Jun '16

08 Jun '16

Fabian Deutsch has submitted this change and it was merged. Change subject: node: Drop hack ...................................................................... node: Drop hack This hack does not work, because th eproblem is authconfig. authconfig is run as a post-installation task, and overriding nsswitch.conf. Because it is not aware of myhostname, it will always not add it to the hosts line. Change-Id: Ic85d77baadabff7357c76ee88b100746499a3c54 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M ovirt-release-master.spec.in 1 file changed, 0 insertions(+), 7 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58835 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic85d77baadabff7357c76ee88b100746499a3c54 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: node: Allow a separate suffix for the placeholder package
by fabiand＠redhat.com 08 Jun '16

08 Jun '16

Fabian Deutsch has submitted this change and it was merged. Change subject: node: Allow a separate suffix for the placeholder package ...................................................................... node: Allow a separate suffix for the placeholder package This is needed to allow snapshot NGN builds in the stable branches Change-Id: I559b25168fcee4b3c11931f064394da44a3e173d Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M ovirt-release-master.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58838 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I559b25168fcee4b3c11931f064394da44a3e173d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0