July 2014 - Engine-commits - oVirt List Archives

Change in ovirt-reports[ovirt-engine-reports-3.4]: packaging: added release notes
by ydary＠redhat.com 08 Jul '14

08 Jul '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: added release notes ...................................................................... packaging: added release notes Change-Id: Ica53b81e43b5191632e4a6357cc6e8f2854ca402 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M ovirt-engine-reports.spec.in 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Shirly Radco: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29701 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ica53b81e43b5191632e4a6357cc6e8f2854ca402 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-reports Gerrit-Branch: ovirt-engine-reports-3.4 Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core, engine: Fix CD-ROM payload in ppc64 VMs
by rgolan＠redhat.com 08 Jul '14

08 Jul '14

Roy Golan has submitted this change and it was merged. Change subject: core, engine: Fix CD-ROM payload in ppc64 VMs ...................................................................... core, engine: Fix CD-ROM payload in ppc64 VMs This change fixes the addressing for payload CD-ROMs in ppc64 VMs, previously there was not a way to address two virtual CD drives at the same time in the sPAPR VSCSI controller. Change-Id: Ieca8008621392b95cb9dcd5eb8ca230233de19dc Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1073540 Signed-off-by: Vitor de Lima <vitor.lima(a)eldorado.org.br> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VmInfoBuilder.java 1 file changed, 14 insertions(+), 6 deletions(-) Approvals: Gustavo Frederico Temple Pedrosa: Verified Roy Golan: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25632 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieca8008621392b95cb9dcd5eb8ca230233de19dc Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vitor de Lima <vitor.lima(a)eldorado.org.br> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gustavo Frederico Temple Pedrosa <gustavo.pedrosa(a)eldorado.org.br> Gerrit-Reviewer: Leonardo Bianconi <leonardo.bianconi(a)eldorado.org.br> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Vitor de Lima <vitor.lima(a)eldorado.org.br> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-reports[ovirt-engine-reports-3.4]: packaging: setup: disable Spring integration in JRS pro
by ydary＠redhat.com 08 Jul '14

08 Jul '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: disable Spring integration in JRS pro ...................................................................... packaging: setup: disable Spring integration in JRS pro This is needed to be compatiable with some versions of JBoss. Change-Id: Ic1e4bd0ece4ae9386ac4f816a89d68ae7dabf4c0 Bug-Url: https://bugzilla.redhat.com/1117190 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> (cherry picked from commit 87f9750c6e25317fab20b4f8faf4c406674fd186) --- A packaging/jasper-war-patches/pro/200_007_web-inf_disable_spring_integration.patch 1 file changed, 13 insertions(+), 0 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29700 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic1e4bd0ece4ae9386ac4f816a89d68ae7dabf4c0 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-reports Gerrit-Branch: ovirt-engine-reports-3.4 Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: Change trigger for VnicProfileWidget ValueChangeEvent
by lvernia＠redhat.com 08 Jul '14

08 Jul '14

Lior Vernia has submitted this change and it was merged. Change subject: webadmin: Change trigger for VnicProfileWidget ValueChangeEvent ...................................................................... webadmin: Change trigger for VnicProfileWidget ValueChangeEvent VnicProfileWidget used to listen on its nameEditor's ValueChangeEvent in order to fire a ValueChangeEvent on its own. However, when the nameEditor's value was changed programmatically from its model, UiCommonEditorVisitor would call an overload of nameEditor.setValue() that would set the value without firing a ValueChangeEvent. This patch registers directly to the model, so that the programmatical change on the model side would trigger a ValueChangeEvent of VnicProfileWidget. Change-Id: I330c38ecedb0df6433491bedaeb7afdc2bd11ae5 Bug-Url: https://bugzilla.redhat.com/1114971 Signed-off-by: Lior Vernia <lvernia(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/vnicProfile/VnicProfileWidget.java 1 file changed, 5 insertions(+), 2 deletions(-) Approvals: Alona Kaplan: Looks good to me, approved Lior Vernia: Verified -- To view, visit http://gerrit.ovirt.org/29526 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I330c38ecedb0df6433491bedaeb7afdc2bd11ae5 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Bind port to agent when port is being activated
by masayag＠redhat.com 08 Jul '14

08 Jul '14

Moti Asayag has submitted this change and it was merged. Change subject: engine: Bind port to agent when port is being activated ...................................................................... engine: Bind port to agent when port is being activated When using the ml2 core plugin, the activated port should be associated with the host on which it has been scheduled. Bug-Url: http://bugzilla.redhat.com/1078862 Change-Id: I0cb5bf95054b6d2711f88a09d374a74f156a4055 Signed-off-by: Moti Asayag <masayag(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/network/vm/ActivateDeactivateVmNicCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/network/NetworkProviderProxy.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/network/openstack/OpenstackNetworkProviderProxy.java M ovirt-engine.spec.in M pom.xml 6 files changed, 41 insertions(+), 15 deletions(-) Approvals: Mike Kolesnik: Looks good to me, approved Omer Frenkel: Looks good to me, but someone else must approve Moti Asayag: Verified -- To view, visit http://gerrit.ovirt.org/28792 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0cb5bf95054b6d2711f88a09d374a74f156a4055 Gerrit-PatchSet: 13 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Fix Dynamic Queries to search all fields for non exist...
by oourfali＠redhat.com 08 Jul '14

08 Jul '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: Fix Dynamic Queries to search all fields for non existing field ...................................................................... core: Fix Dynamic Queries to search all fields for non existing field Make sure that when searching for a field that doesn't appear we search the _with_tags table. Change-Id: I1aef8977331f9b3bd1d5003fbd9c7104b176eb92 Bug-Url: https://bugzilla.redhat.com/1114992 Signed-off-by: lzelkha(a)redhat.com <lzelkha(a)redhat.com> --- M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/SyntaxChecker.java M backend/manager/modules/searchbackend/src/test/java/org/ovirt/engine/core/searchbackend/SyntaxCheckerTest.java 2 files changed, 53 insertions(+), 10 deletions(-) Approvals: Liran Zelkha: Verified Yair Zaslavsky: Looks good to me, approved Oved Ourfali: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/29444 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1aef8977331f9b3bd1d5003fbd9c7104b176eb92 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: remove redundant synchronization (CoCo cleanup)
by oourfali＠redhat.com 08 Jul '14

08 Jul '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: remove redundant synchronization (CoCo cleanup) ...................................................................... core: remove redundant synchronization (CoCo cleanup) a method which is executed by our quartz in fixed delay can only be executed by one thread at each given time, because before executing it we remove it from the quartz pool and when it ends we put it back. so no need to synchronize anything for protecting from parallel executions of the same critical section Change-Id: I588de14e21f19ef59781b382b2450b8723635667 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CommandExecutor.java 1 file changed, 5 insertions(+), 10 deletions(-) Approvals: Ravi Nori: Looks good to me, but someone else must approve Arik Hadas: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29522 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I588de14e21f19ef59781b382b2450b8723635667 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: ui: Fixing hostPopupView tab indexes
by oourfali＠redhat.com 07 Jul '14

07 Jul '14

Oved Ourfali has submitted this change and it was merged. Change subject: ui: Fixing hostPopupView tab indexes ...................................................................... ui: Fixing hostPopupView tab indexes Change-Id: Ie183a178c86cbdfc58e52baf75700d44e5a40934 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1115045 Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostPopupView.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Yaniv Bronhaim: Verified Daniel Erez: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29626 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie183a178c86cbdfc58e52baf75700d44e5a40934 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Warning instead of error if kdump not configured
by oourfali＠redhat.com 07 Jul '14

07 Jul '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: Warning instead of error if kdump not configured ...................................................................... core: Warning instead of error if kdump not configured Show warning instead of error if kdump is not configured properly on host. Change-Id: I51f21f74198a2e81222363c31e399139844a06ed Bug-Url: https://bugzilla.redhat.com/1114965 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Martin Peřina: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29663 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I51f21f74198a2e81222363c31e399139844a06ed Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-node-plugin-vdsm[master]: packaging: Depend on ovirt-host-deploy-offline >= 1.3.0
by dougsland＠redhat.com 07 Jul '14

07 Jul '14

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: packaging: Depend on ovirt-host-deploy-offline >= 1.3.0 ...................................................................... packaging: Depend on ovirt-host-deploy-offline >= 1.3.0 Adds dependency on ovirt-host-deploy-offline >= 1.3.0 to fetch needed dependencies for kdump integration. Change-Id: I28e0f11269bdc26a24a9892b17cfef557b02e42b Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M ovirt-node-plugin-vdsm.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29659 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I28e0f11269bdc26a24a9892b17cfef557b02e42b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0