December 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: VDSError - removing the unused args member
by Code Review 15 Dec '16

15 Dec '16

>From Moti Asayag <masayag(a)redhat.com>: Moti Asayag has submitted this change and it was merged. Change subject: core: VDSError - removing the unused args member ...................................................................... core: VDSError - removing the unused args member Change-Id: I84e0c35436781bcbed77b7acfba97cca53a6768a Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VDSError.java 1 file changed, 0 insertions(+), 11 deletions(-) Approvals: Jenkins CI: Passed CI tests Freddy Rolland: Looks good to me, but someone else must approve Moti Asayag: Looks good to me, approved Liron Aravot: Verified -- To view, visit https://gerrit.ovirt.org/68406 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I84e0c35436781bcbed77b7acfba97cca53a6768a Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Make mock_cleanup.sh safer about un-mouting
by Code Review 15 Dec '16

15 Dec '16

>From Eyal Edri <eedri(a)redhat.com>: Eyal Edri has submitted this change and it was merged. Change subject: Make mock_cleanup.sh safer about un-mouting ...................................................................... Make mock_cleanup.sh safer about un-mouting mock_cleanup.sh used to fail everything when umounting failed, this patch addes a retry to un-mouting and also make it not fail if un-mounting failed simply because the thing we wanted to um-mount is not mounted. Jira-Ticket: https://ovirt-jira.atlassian.net/browse/OVIRT-937 Signed-off-by: Barak Korren <bkorren(a)redhat.com> Change-Id: Ic82506abe1ffffc876a283a45f344f09842a0e1e --- M jobs/confs/shell-scripts/mock_cleanup.sh 1 file changed, 29 insertions(+), 18 deletions(-) Approvals: Nir Soffer: Looks good to me Eyal Edri: Looks good to me, approved; Ready for merge Barak Korren: Verified; Ready for review Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/68430 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic82506abe1ffffc876a283a45f344f09842a0e1e Gerrit-PatchSet: 4 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: David Caro <david(a)dcaro.es> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nadav Goldin <ngoldin(a)redhat.com> Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-ha[v2.0.z]: he: Prevent VM from migrating when there is no host for it
by Code Review 15 Dec '16

15 Dec '16

>From Martin Sivák <msivak(a)redhat.com>: Martin Sivák has submitted this change and it was merged. Change subject: he: Prevent VM from migrating when there is no host for it ...................................................................... he: Prevent VM from migrating when there is no host for it With this patch VM migration during transition to LocalMaintenance mode is prevented if all other HE hosts are already in LocalMaintenane mode or not capable to run th VM. Change-Id: Idf5030aeada317a01b68b825f74eba57bc6d2c56 Bug-Url: https://bugzilla.redhat.com/1394570 Signed-off-by: Denis Chaplygin <dchaplyg(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/state_machine.py M ovirt_hosted_engine_ha/agent/states.py 2 files changed, 6 insertions(+), 2 deletions(-) Approvals: Denis Chaplygin: Verified Martin Sivák: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/68436 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idf5030aeada317a01b68b825f74eba57bc6d2c56 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: v2.0.z Gerrit-Owner: Denis Chaplygin <dchaplyg(a)redhat.com> Gerrit-Reviewer: Denis Chaplygin <dchaplyg(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: spec: add require on collectd-disk
by Code Review 15 Dec '16

15 Dec '16

>From Sandro Bonazzola <sbonazzo(a)redhat.com>: Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: spec: add require on collectd-disk ...................................................................... packaging: spec: add require on collectd-disk add require on collectd-disk now required by ovirt-host-deploy. Change-Id: I6ff2705fc02ea91b0bd89731a7d66f47ba8ba976 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-hosted-engine-setup.spec.in 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/68519 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6ff2705fc02ea91b0bd89731a7d66f47ba8ba976 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: OVIRT-645 remove repoproxy for Fedora and EL6 x86_64
by Code Review 15 Dec '16

15 Dec '16

>From Anton Marchukov <amarchuk(a)redhat.com>: Anton Marchukov has submitted this change and it was merged. Change subject: OVIRT-645 remove repoproxy for Fedora and EL6 x86_64 ...................................................................... OVIRT-645 remove repoproxy for Fedora and EL6 x86_64 This patch removes repoproxy from Fedora 23,24,25 and CentOS 6 mock configs in favour of usning a trimmed down mirrorlist via SQUID. Also fixes a typo in CentOS 7 definitions for a mirror. Change-Id: Id2f7e7b68c9f726a9228bb20fc4c52a28fead057 Signed-off-by: Evgheni Dereveanchin <ederevea(a)redhat.com> --- M mock_configs/epel-6-x86_64_proxied.cfg M mock_configs/epel-7-x86_64_proxied.cfg M mock_configs/fedora-23-x86_64_proxied.cfg M mock_configs/fedora-24-x86_64_proxied.cfg M mock_configs/fedora-25-x86_64_proxied.cfg 5 files changed, 75 insertions(+), 29 deletions(-) Approvals: Anton Marchukov: Looks good to me, approved; Ready for merge Jenkins CI: Passed CI tests Evgheni Dereveanchin: Verified -- To view, visit https://gerrit.ovirt.org/67595 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id2f7e7b68c9f726a9228bb20fc4c52a28fead057 Gerrit-PatchSet: 3 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Evgheni Dereveanchin <ederevea(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Evgheni Dereveanchin <ederevea(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Fixes in the amend operation.
by Code Review 15 Dec '16

15 Dec '16

>From Maor Lipchuk <mlipchuk(a)redhat.com>: Maor Lipchuk has submitted this change and it was merged. Change subject: core: Fixes in the amend operation. ...................................................................... core: Fixes in the amend operation. 1. Adding update of qcow compat in end command. 2. Using storage pool id from the parameters. Change-Id: I88cda9895e6e188dcb7d7c0947a930320be2d433 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/AmendImageGroupVolumesCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/AmendVolumeCommand.java 2 files changed, 18 insertions(+), 9 deletions(-) Approvals: Daniel Erez: Looks good to me, approved Jenkins CI: Passed CI tests Maor Lipchuk: Verified -- To view, visit https://gerrit.ovirt.org/68493 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I88cda9895e6e188dcb7d7c0947a930320be2d433 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-host-deploy[ovirt-host-deploy-1.6]: metrics: Install collectd-disk
by Code Review 15 Dec '16

15 Dec '16

>From Sandro Bonazzola <sbonazzo(a)redhat.com>: Sandro Bonazzola has submitted this change and it was merged. Change subject: metrics: Install collectd-disk ...................................................................... metrics: Install collectd-disk Bug-Url: https://bugzilla.redhat.com/1371530 Change-Id: I222c2a88e493c0b2d3429caf8243bc2590de4d62 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> (cherry picked from commit b97a668c903564edc9b03d3fe5e2d38bf53aaff7) --- M src/plugins/ovirt-host-deploy/collectd/packages.py 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/68515 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I222c2a88e493c0b2d3429caf8243bc2590de4d62 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: ovirt-host-deploy-1.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Affinity Rules Enforcer VM to hosts support
by Code Review 15 Dec '16

15 Dec '16

>From Martin Sivák <msivak(a)redhat.com>: Martin Sivák has submitted this change and it was merged. Change subject: core: Affinity Rules Enforcer VM to hosts support ...................................................................... core: Affinity Rules Enforcer VM to hosts support Added support for VM to hosts affinity to the existing Affinity rule enforcer. Change-Id: I90bab988a93cc5d45f9e10594d0f8356866402c4 Bug-Url: https://bugzilla.redhat.com/1404660 Signed-off-by: Yanir Quinn <yquinn(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/arem/AffinityRulesEnforcer.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/arem/AffinityRulesEnforcerTest.java 2 files changed, 306 insertions(+), 20 deletions(-) Approvals: Martin Sivák: Looks good to me, approved Jenkins CI: Passed CI tests Yanir Quinn: Verified -- To view, visit https://gerrit.ovirt.org/68104 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I90bab988a93cc5d45f9e10594d0f8356866402c4 Gerrit-PatchSet: 12 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yanir Quinn <yquinn(a)redhat.com> Gerrit-Reviewer: Andrej Krejcir <akrejcir(a)redhat.com> Gerrit-Reviewer: Denis Chaplygin <dchaplyg(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: Yanir Quinn <yquinn(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-host-deploy[master]: metrics: Install collectd-disk
by Code Review 15 Dec '16

15 Dec '16

>From Sandro Bonazzola <sbonazzo(a)redhat.com>: Sandro Bonazzola has submitted this change and it was merged. Change subject: metrics: Install collectd-disk ...................................................................... metrics: Install collectd-disk Bug-Url: https://bugzilla.redhat.com/1371530 Change-Id: I222c2a88e493c0b2d3429caf8243bc2590de4d62 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M src/plugins/ovirt-host-deploy/collectd/packages.py 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/68514 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I222c2a88e493c0b2d3429caf8243bc2590de4d62 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[master]: Update to model 4.1.24
by Code Review 15 Dec '16

15 Dec '16

>From Juan Hernandez <juan.hernandez(a)redhat.com>: Juan Hernandez has submitted this change and it was merged. Change subject: Update to model 4.1.24 ...................................................................... Update to model 4.1.24 The more relevant changes in this new version of the model are the following: * Added new `v4` value to the `StorageFormat` enum. Change-Id: Ic736c73910669ce257afca2e238bd7fe7b8978bb Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M pom.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Ondra Machacek: Verified Juan Hernandez: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/68512 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic736c73910669ce257afca2e238bd7fe7b8978bb Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0