- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: Revert "core: Prevent StopVm to interleave with other VM loc...
by ofrenkel＠redhat.com 11 Mar '14

11 Mar '14

Omer Frenkel has submitted this change and it was merged. Change subject: Revert "core: Prevent StopVm to interleave with other VM locked actions" ...................................................................... Revert "core: Prevent StopVm to interleave with other VM locked actions" This reverts commit 37411acc82c03ba8e3150e5d9b4aad2e6d39a414. Change-Id: I0c12f4e2855e9ddfb6d696167b20352914825a9a Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ShutdownVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/StopVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/StopVmCommandBase.java 3 files changed, 1 insertion(+), 20 deletions(-) Approvals: Roy Golan: Verified Omer Frenkel: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25567 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0c12f4e2855e9ddfb6d696167b20352914825a9a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: userportal, webadmin: Prevent migration on the frontend
by tjelinek＠redhat.com 11 Mar '14

11 Mar '14

Tomas Jelinek has submitted this change and it was merged. Change subject: userportal, webadmin: Prevent migration on the frontend ...................................................................... userportal, webadmin: Prevent migration on the frontend This patch transparently disables migration when creating VMs on architectures that do not support it. This includes pinning the created VM to a host and forcing the "Do not migrate" resilience policy. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1057178 Change-Id: I4e6f82dd8906fe8aa3d6c9e28eaf0e155feef51c Signed-off-by: Vitor de Lima <vitor.lima(a)eldorado.org.br> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java 3 files changed, 63 insertions(+), 1 deletion(-) Approvals: Vitor de Lima: Verified Tomas Jelinek: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21523 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4e6f82dd8906fe8aa3d6c9e28eaf0e155feef51c Gerrit-PatchSet: 15 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vitor de Lima <vitor.lima(a)eldorado.org.br> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(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: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(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-engine[master]: userportal, webadmin: Prevent snapshot in ppc64
by tjelinek＠redhat.com 11 Mar '14

11 Mar '14

Tomas Jelinek has submitted this change and it was merged. Change subject: userportal, webadmin: Prevent snapshot in ppc64 ...................................................................... userportal, webadmin: Prevent snapshot in ppc64 This change disables memory snapshot in the frontend for clusters with architecture that doesn't support it. The option for create memory snapshot is now hidden for these cases. All logic was added in conjunction with the snapshot validation from the parameter ConfigurationValues.MemorySnapshotSupported. Change-Id: I148f0a9356086b31e66d4f5dd6eab6afc642cada Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1061774 Signed-off-by: Leonardo Bianconi <leonardo.bianconi(a)eldorado.org.br> Signed-off-by: Vitor de Lima <vitor.lima(a)eldorado.org.br> Signed-off-by: Gustavo Pedrosa <gustavo.pedrosa(a)eldorado.org.br> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmSnapshotCreatePopupWidget.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/vm/VmSnapshotListModelTable.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmSnapshotListModel.java 4 files changed, 45 insertions(+), 16 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Leonardo Bianconi: Verified -- To view, visit http://gerrit.ovirt.org/21658 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I148f0a9356086b31e66d4f5dd6eab6afc642cada Gerrit-PatchSet: 13 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Leonardo Bianconi <leonardo.bianconi(a)eldorado.org.br> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(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: Liron Ar <laravot(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Vitor de Lima <vitor.lima(a)eldorado.org.br> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: liron aravot <liron.aravot(a)gmail.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.4.0]: build: post ovirt-engine-3.4.0_rc
by sbonazzo＠redhat.com 11 Mar '14

11 Mar '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: post ovirt-engine-3.4.0_rc ...................................................................... build: post ovirt-engine-3.4.0_rc Change-Id: I85d09a8cf5621e9192cfaa0f48579fdabb33538f Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-engine.spec.in M version.mak 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25612 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I85d09a8cf5621e9192cfaa0f48579fdabb33538f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.4.0]: build: ovirt-engine-3.4.0_rc
by sbonazzo＠redhat.com 11 Mar '14

11 Mar '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-engine-3.4.0_rc ...................................................................... build: ovirt-engine-3.4.0_rc Change-Id: I97801be267f49059b1cebab0fea31c66c151a3a1 Bug-Url: https://bugzilla.redhat.com/1024889 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-engine.spec.in M version.mak 2 files changed, 10 insertions(+), 7 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25610 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I97801be267f49059b1cebab0fea31c66c151a3a1 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in otopi[master]: packaging: post otopi-1.2.0_rc3
by Alon Bar-Lev 11 Mar '14

11 Mar '14

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: post otopi-1.2.0_rc3 ...................................................................... packaging: post otopi-1.2.0_rc3 Change-Id: I4c24686c90ebb8c0dd93c1bb057d9aeaf01a868a Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M configure.ac 2 files changed, 5 insertions(+), 2 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25616 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4c24686c90ebb8c0dd93c1bb057d9aeaf01a868a Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in otopi[master]: packaging: otopi-1.2.0_rc3
by Alon Bar-Lev 11 Mar '14

11 Mar '14

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: otopi-1.2.0_rc3 ...................................................................... packaging: otopi-1.2.0_rc3 Change-Id: I68d99823addf602f4281a3b5a84768081b70401f Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M configure.ac M otopi.spec.in 3 files changed, 6 insertions(+), 3 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25615 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I68d99823addf602f4281a3b5a84768081b70401f Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.4]: packaging: setup: display only top level setup packages
by iheim＠redhat.com 11 Mar '14

11 Mar '14

Itamar Heim has submitted this change and it was merged. Change subject: packaging: setup: display only top level setup packages ...................................................................... packaging: setup: display only top level setup packages currently we print a message with entire transaction content of modified packages for setup packages. better to show only setup packages in price of some performance. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1075043 Change-Id: I3a1f27090aa362882047007416aacff1cb5dbc4c Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-common/distro-rpm/packages.py 1 file changed, 6 insertions(+), 12 deletions(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25611 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3a1f27090aa362882047007416aacff1cb5dbc4c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-reports[ovirt-3.3]: packaging: added release notes
by ydary＠redhat.com 11 Mar '14

11 Mar '14

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

1 0

Change in ovirt-engine[ovirt-engine-3.4]: engine: Remove port if hot plug new ext. nic fails
by iheim＠redhat.com 11 Mar '14

11 Mar '14

Itamar Heim has submitted this change and it was merged. Change subject: engine: Remove port if hot plug new ext. nic fails ...................................................................... engine: Remove port if hot plug new ext. nic fails If a new vNIC is added on an external network, and this vNIC is also hot plugged as part of this flow then a corresponding entity will be created on the external provider. If the aforementioned action fails, the vNIC will be removed from the DB (by compensation mechanism) but not from the external provider, leaving garbage forever on it. In this fix, a new vNIC that fails to hot plug will also be removed from the external provider. At this point, there is no need to remove an existing vNIC should the hot plug fail, as the entity that was created on the external provider should be removed only when the vNIC is being removed, or the network is not used by the vNIC anymore. Change-Id: Iddeac63a47286e9b83ac02fb2e55f7dd56092805 Bug-Url: https://bugzilla.redhat.com/987814 Signed-off-by: Mike Kolesnik <mkolesni(a)redhat.com> (cherry picked from commit c3fcf1485b5b15e4b9509b910b24202f39ea594c) --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/vm/AbstractVmInterfaceCommand.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/network/vm/AddVmInterfaceCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/vm/UpdateVmInterfaceCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/ActivateDeactivateVmNicParameters.java 5 files changed, 58 insertions(+), 21 deletions(-) Approvals: Mike Kolesnik: Verified Moti Asayag: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25603 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iddeac63a47286e9b83ac02fb2e55f7dd56092805 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0