November 2012 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: Revert "Revert "core: errors during preview of diskless snap...
by yzaslavs＠redhat.com 14 Nov '12

14 Nov '12

Yair Zaslavsky has submitted this change and it was merged. Change subject: Revert "Revert "core: errors during preview of diskless snapshots"" ...................................................................... Revert "Revert "core: errors during preview of diskless snapshots"" This reverts commit a5b2196fdf9299ac774588aaaa68415e51ee509b --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/TryBackToAllSnapshotsOfVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java 2 files changed, 16 insertions(+), 12 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9242 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5c9fd926e61e860f4af9ff460c6657cf81b1fc85 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: Add/Edit Network dialog bugs with message and vm n...
by alkaplan＠redhat.com 14 Nov '12

14 Nov '12

Alona Kaplan has submitted this change and it was merged. Change subject: webadmin: Add/Edit Network dialog bugs with message and vm network ...................................................................... webadmin: Add/Edit Network dialog bugs with message and vm network 1. Edit management network dialog didn't show correct message on top of it. 2. New Network dialog (from Cluster->Logical Network) didn't show correct message on top of it. 3. Vm network was always checked in Edit Network dialog (even if it is not vm network). Change-Id: I29fb26f2c62f501e6c08c706bba1174f91466228 Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterNetworkListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/EditNetworkModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/NetworkModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/NewNetworkModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/popup/AbstractNetworkPopupPresenterWidget.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/popup/cluster/NewClusterNetworkPopupPresenterWidget.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/AbstractNetworkPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/NewClusterNetworkPopupView.java 8 files changed, 207 insertions(+), 176 deletions(-) Approvals: Alona Kaplan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9226 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I29fb26f2c62f501e6c08c706bba1174f91466228 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>

1 0

Change in ovirt-engine[master]: Revert "core: errors during preview of diskless snapshots"
by mkublin＠redhat.com 14 Nov '12

14 Nov '12

Michael Kublin has submitted this change and it was merged. Change subject: Revert "core: errors during preview of diskless snapshots" ...................................................................... Revert "core: errors during preview of diskless snapshots" This reverts commit 383837efbdee6651e2032c0d6723078c099fb66d. The following commit introduced performance regression which will influence the performance of all system. Regression is: XML RPC call inside transaction Change-Id: I696457016d2b475e42be2c67b55fdf12b12d5193 Signed-off-by: Michael Kublin <mkublin(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/TryBackToAllSnapshotsOfVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java 2 files changed, 12 insertions(+), 16 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9240 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I696457016d2b475e42be2c67b55fdf12b12d5193 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Fixed bll tests logging
by yzaslavs＠redhat.com 14 Nov '12

14 Nov '12

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Fixed bll tests logging ...................................................................... core: Fixed bll tests logging Log4j configuration contains outdated package names from before oVirt was open sourced. This patch fixes those issues, and uses the correct org.ovirt.core package. Change-Id: Ia2370849ca588153f97be5884ee62bde0c63be55 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/resources/log4j.xml 1 file changed, 13 insertions(+), 13 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9235 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia2370849ca588153f97be5884ee62bde0c63be55 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Introducing saveOrUpdate to AsyncTasks
by yzaslavs＠redhat.com 14 Nov '12

14 Nov '12

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Introducing saveOrUpdate to AsyncTasks ...................................................................... core: Introducing saveOrUpdate to AsyncTasks This performance improvement to the add or update of tasks is introduced - No more querying to check the db if tasks exists and according to it deciding if to save or update, but performing all this logic at DB side. Bug-Url: https://bugzilla.redhat.com/873546 Change-Id: Iebfc85569ba1aa8bd840f7239f83b7f921a4bd7e Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/dbscripts/async_tasks_sp.sql M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/AsyncTaskUtils.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/AsyncTaskDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/AsyncTaskDAODbFacadeImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/AsyncTaskDAOTest.java 5 files changed, 84 insertions(+), 7 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9232 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iebfc85569ba1aa8bd840f7239f83b7f921a4bd7e Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: re-run mechanism improvements
by ofrenkel＠redhat.com 14 Nov '12

14 Nov '12

Omer Frenkel has submitted this change and it was merged. Change subject: engine: re-run mechanism improvements ...................................................................... engine: re-run mechanism improvements - RunVmCommandBase#rerun method no longer invokes rerunInternal method within a different thread because there is no need for a different thread when it is called from the command itself. the code of rerunInternal method was moved to rerun method without creating a new thread to run it instead. when rerun method is invoked by VdsEventListener, its code should still run in a different thread so it is called from a new thread in VdsEventListener#rerun method. - adjust the rerun mechanism such that the logs & events would reflect the process better: * on re-run, log the results of the previous run before checking whether the maximum rerun attempts were exceeded, so that it would be logged every time. * add a missing log call after exceeding the maximum re-run attempts to log a message that indicates the command failed. * make sure that when a command that is being rerun is failed due to can-do-action check, the failure event will appear only once. Change-Id: I557636a94d74772635102c6a33dc923d91332a3e Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmToServerCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsEventListener.java 4 files changed, 30 insertions(+), 23 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9184 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I557636a94d74772635102c6a33dc923d91332a3e 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: Roy Golan <rgolan(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: UpdateVmDisk - clear snapshot_id for shareable disk
by derez＠redhat.com 14 Nov '12

14 Nov '12

Daniel Erez has submitted this change and it was merged. Change subject: core: UpdateVmDisk - clear snapshot_id for shareable disk ...................................................................... core: UpdateVmDisk - clear snapshot_id for shareable disk UpdateVmDiskCommand should nullify vm_snapshot_id when the disk is marked as shareable. Change-Id: I259759658c1baea8001fa151a5e861cecd8e794a Bug-Url: https://bugzilla.redhat.com/872945 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmDiskCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/UpdateVmDiskCommandTest.java 2 files changed, 27 insertions(+), 11 deletions(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9231 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I259759658c1baea8001fa151a5e861cecd8e794a Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Execution messages for Gluster Volume Actions (#8561...
by emesika＠redhat.com 14 Nov '12

14 Nov '12

Eli Mesika has submitted this change and it was merged. Change subject: engine: Execution messages for Gluster Volume Actions (#856102) ...................................................................... engine: Execution messages for Gluster Volume Actions (#856102) https://bugzilla.redhat.com/856102 Execution messages added for all Gluster Volume related actions. Change-Id: I93bb7b8b1f1ae4987c0887307bd373a28e9ad3d1 Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- M backend/manager/dbscripts/create_functions.sql M backend/manager/modules/dal/src/main/resources/bundles/ExecutionMessages.properties 2 files changed, 13 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7967 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I93bb7b8b1f1ae4987c0887307bd373a28e9ad3d1 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: fix imports in MoveImageGroupVDSCommand
by lhornyak＠redhat.com 14 Nov '12

14 Nov '12

Laszlo Hornyak has submitted this change and it was merged. Change subject: core: fix imports in MoveImageGroupVDSCommand ...................................................................... core: fix imports in MoveImageGroupVDSCommand Change-Id: Ia313b6343d1e554476662dfc604d59d672088401 Signed-off-by: Sharad Mishra <snmishra(a)linux.vnet.ibm.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/MoveImageGroupVDSCommand.java 1 file changed, 4 insertions(+), 3 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9094 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia313b6343d1e554476662dfc604d59d672088401 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sharad Mishra <snmishra(a)linux.vnet.ibm.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Sharad Mishra <snmishra(a)linux.vnet.ibm.com>

1 0

Change in ovirt-engine[master]: engine: prevent NPE in clusters tab
by shavivi＠redhat.com 14 Nov '12

14 Nov '12

Shahar Havivi has submitted this change and it was merged. Change subject: engine: prevent NPE in clusters tab ...................................................................... engine: prevent NPE in clusters tab Change-Id: If8f344b9144880b3ecd755030594d28c12912043 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=871293 Signed-off-by: Shahar Havivi <shavivi(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/cluster/SubTabClusterHostView.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Shahar Havivi: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9181 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If8f344b9144880b3ecd755030594d28c12912043 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Sharad Mishra <snmishra(a)linux.vnet.ibm.com>

1 0

Change in ovirt-engine[master]: webadmin: layout: fix layout in new storage domain - iscsi u...
by derez＠redhat.com 13 Nov '12

13 Nov '12

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: layout: fix layout in new storage domain - iscsi username ...................................................................... webadmin: layout: fix layout in new storage domain - iscsi username Change-Id: I881bb2bf6d79a5c04d87313dd038167db3c1ea5e Bug-url: https://bugzilla.redhat.com/867428 Signed-off-by: Shahar Havivi <shavivi(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/storage/IscsiDiscoverTargetsView.ui.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/StoragePopupView.ui.xml 2 files changed, 15 insertions(+), 12 deletions(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9185 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I881bb2bf6d79a5c04d87313dd038167db3c1ea5e Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>

1 0

Change in ovirt-engine[master]: core:Force flag should be set from parameters
by mlipchuk＠redhat.com 13 Nov '12

13 Nov '12

Maor Lipchuk has submitted this change and it was merged. Change subject: core:Force flag should be set from parameters ...................................................................... core:Force flag should be set from parameters When import VM or VM Template, we should set the force flag in the REST, and also set the parameter in the engine before calling MoveOrCopyImageGroupCommand. Change-Id: Ie3a8de587e9a9384a2cf51dc7435967d46d3d9fd Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/876139 --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmTemplateCommand.java M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata_v-3.1.yaml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainTemplateResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainVmResource.java 5 files changed, 6 insertions(+), 4 deletions(-) Approvals: Maor Lipchuk: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9224 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie3a8de587e9a9384a2cf51dc7435967d46d3d9fd Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Correct migration related events
by ofrenkel＠redhat.com 13 Nov '12

13 Nov '12

Omer Frenkel has submitted this change and it was merged. Change subject: engine: Correct migration related events ...................................................................... engine: Correct migration related events The misleading events: "Migration of VM xp_desktop from Host knight failed. Error: Migration succeeded" "VM oo is down. Exit message: Migration succeeded" are no longer printed when the migration process succeeds Change-Id: I0b92611b423ee3f2336816873505566c41a17b0b Bug-Url: https://bugzilla.redhat.com/836780 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java 1 file changed, 26 insertions(+), 11 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9199 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0b92611b423ee3f2336816873505566c41a17b0b Gerrit-PatchSet: 3 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: Roy Golan <rgolan(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin, engine: Add validation for snapshot description le...
by masayag＠redhat.com 13 Nov '12

13 Nov '12

Moti Asayag has submitted this change and it was merged. Change subject: webadmin, engine: Add validation for snapshot description length ...................................................................... webadmin, engine: Add validation for snapshot description length Add validation which checks that the length of the snapshot description does not exceed 4000 (GENERAL_MAX_SIZE) characters at the webadmin dialog for creating snapshot Add validation which checks that the length of the snapshot description does not exceed GENERAL_MAX_SIZE for CreateAllSnapshhotsFromVmCommand in the engine Change-Id: I780eec76b6eb576d28b3650f4d34384594a8044d Bug-Url: https://bugzilla.redhat.com/871029 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/CreateAllSnapshotsFromVmParameters.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/SnapshotModel.java M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 7 files changed, 15 insertions(+), 1 deletion(-) Approvals: Moti Asayag: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9011 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I780eec76b6eb576d28b3650f4d34384594a8044d Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(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: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: exception on remove network from main tab.
by alkaplan＠redhat.com 13 Nov '12

13 Nov '12

Alona Kaplan has submitted this change and it was merged. Change subject: webadmin: exception on remove network from main tab. ...................................................................... webadmin: exception on remove network from main tab. ClassCastException when removing network from Network Main tab. Tried to cast NetworkView to Network. Change-Id: I43cea740c8d3c104b7c3ae43772b7fad473ad7fe Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/RemoveNetworksModel.java 1 file changed, 27 insertions(+), 12 deletions(-) Approvals: Alona Kaplan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9210 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I43cea740c8d3c104b7c3ae43772b7fad473ad7fe Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>

1 0

Change in ovirt-engine[master]: core:Hosts from another data center ...(#874660)
by emesika＠redhat.com 13 Nov '12

13 Nov '12

Eli Mesika has submitted this change and it was merged. Change subject: core:Hosts from another data center ...(#874660) ...................................................................... core:Hosts from another data center ...(#874660) Hosts from another data center displayed when clicked on Hosts in tree view for cluster names with a single difference. Search Engine is using currently the ILIKE keyword to enable case-insensitive searches. Postgres treats the character '_' in a value when using LIKE/ILIKE teh same way as '?' is used for regular expressions. (i.e. match any single value) This patch looks for the character '_' in values passed to the search engine and proceed then with a backslash such that they will be treated as a regular character and Search Engine will return the correct result. Change-Id: I2a5e9946f7795827d1abd50f05e641d90802e1e8 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/SyntaxChecker.java 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9207 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2a5e9946f7795827d1abd50f05e641d90802e1e8 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: fix in GlusterManager
by ofrenkel＠redhat.com 13 Nov '12

13 Nov '12

Omer Frenkel has submitted this change and it was merged. Change subject: engine: fix in GlusterManager ...................................................................... engine: fix in GlusterManager A parallel change in the GlusterServersListVDSCommand had changed the return type from Set to List. Changing GlusterManager accordingly. Change-Id: Iefd20756dee267592dfb70754d14f60938973fe6 Signed-off-by: Shireesh Anjal <sanjal(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterManager.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/GlusterManagerTest.java 2 files changed, 11 insertions(+), 13 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9135 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iefd20756dee267592dfb70754d14f60938973fe6 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shireesh Anjal <sanjal(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Sharad Mishra <snmishra(a)linux.vnet.ibm.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Refactor statements with ternary operators
by masayag＠redhat.com 12 Nov '12

12 Nov '12

Moti Asayag has submitted this change and it was merged. Change subject: engine: Refactor statements with ternary operators ...................................................................... engine: Refactor statements with ternary operators Re-format complex statements with ternary operators in MigrateVmCommand#getAuditLogTypeValue and RunVmCommand#getAuditLogTypeValue methods so that they would be more readable. Change-Id: I3d28bd2813a69ab4968ac76927f892fccad20164 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java 2 files changed, 20 insertions(+), 11 deletions(-) Approvals: Moti Asayag: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9186 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3d28bd2813a69ab4968ac76927f892fccad20164 Gerrit-PatchSet: 3 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: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sharad Mishra <snmishra(a)linux.vnet.ibm.com>

1 0

Change in ovirt-engine[master]: core:Fix NPE when import VM with clone
by mlipchuk＠redhat.com 12 Nov '12

12 Nov '12

Maor Lipchuk has submitted this change and it was merged. Change subject: core:Fix NPE when import VM with clone ...................................................................... core:Fix NPE when import VM with clone The following patch should fix an NPE which occurs when importing a cloned VM or a cloned VM template, which does not exists in the Data Center. This exception is caused when trying to copy each disk, and use the property WipeAfterDelete from the image in the DB, since the disk does not exists in the DB, we get an NPE. The proposed fix is to use getDestinationDiskImage to get the WipeAfterDelete value, instead use the original image with getImage(). Change-Id: I47b8f82a2bd19320d9953df54f9aa16e4ef9200f Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> Related to Bug-Url: https://bugzilla.redhat.com/862303 --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MoveOrCopyImageGroupCommand.java 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Daniel Erez: Looks good to me, approved Maor Lipchuk: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9176 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I47b8f82a2bd19320d9953df54f9aa16e4ef9200f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: Added version dependency for webadmin-portal pack...
by juan.hernandez＠redhat.com 12 Nov '12

12 Nov '12

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: Added version dependency for webadmin-portal package ...................................................................... packaging: Added version dependency for webadmin-portal package Change-Id: I5876c46e9709703f8ae3ff76ef0df9d608d8f45c Signed-off-by: Alex Lourie <alourie(a)redhat.com> --- M packaging/fedora/spec/ovirt-engine.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9127 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5876c46e9709703f8ae3ff76ef0df9d608d8f45c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: NPEs in AddVmTemplate end methods after engine crash
by mlipchuk＠redhat.com 12 Nov '12

12 Nov '12

Maor Lipchuk has submitted this change and it was merged. Change subject: core: NPEs in AddVmTemplate end methods after engine crash ...................................................................... core: NPEs in AddVmTemplate end methods after engine crash AddVmTemplateCommand endSuccesfully() method had 2 NPEs after engine crash occrured. 1. VmTemplateId wasn't set after loading the async task info because of wrong setter name 2. getVm() returned null because vm id is not set per the parent command parameters. Change-Id: I12c40cd3736a0516473de223ec84301f84983d3b Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmTemplateCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/AddVmTemplateParameters.java 2 files changed, 9 insertions(+), 3 deletions(-) Approvals: Maor Lipchuk: Looks good to me, approved Vered Volansky: Looks good to me, but someone else must approve Liron Aravot: Verified -- To view, visit http://gerrit.ovirt.org/9106 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I12c40cd3736a0516473de223ec84301f84983d3b Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: compensate on wrong object leads to NPE
by mlipchuk＠redhat.com 12 Nov '12

12 Nov '12

Maor Lipchuk has submitted this change and it was merged. Change subject: core: compensate on wrong object leads to NPE ...................................................................... core: compensate on wrong object leads to NPE Compensation during unlock vm is performed on the vm itself and not on the vm dynamic data, which causes to NPE during compensation process as the defined DAO for the status update is per vm dynamic. Change-Id: I9e10cbec65d0d9c8e66794a9fecf2e59facfdb45 Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Maor Lipchuk: Looks good to me, approved Liron Aravot: Verified -- To view, visit http://gerrit.ovirt.org/9074 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9e10cbec65d0d9c8e66794a9fecf2e59facfdb45 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Sharad Mishra <snmishra(a)linux.vnet.ibm.com>

1 0

Change in ovirt-engine[master]: core: errors during preview of diskless snapshots
by mlipchuk＠redhat.com 12 Nov '12

12 Nov '12

Maor Lipchuk has submitted this change and it was merged. Change subject: core: errors during preview of diskless snapshots ...................................................................... core: errors during preview of diskless snapshots 1. When creating a diskless snapshot, then adding a disk to the vm and later on try to preview that snapshots an NPE occurs within ovf writer. The exception occurs because of outdated disk information contained within the vm - the problem wasn't fixed locally but globally to prevent it from occuring in other flows as well. VmHandler.updateDisksFromDb() should clear the current disks of the vm otherwise we might get corrupted disk information for the vm (by adding the disks information from the DB while not clearing the present information). 2. When trying to run diskless snapshots exception can occur for various reason in VmHandler.updateVmInSpm(). When such exception occurs the operation should be rolled back (in the bug scenario - it wasn't as there were db updates that were executed with no transaction which led to partially rollback). Change-Id: I6e8edc74bc34676f526dfd24d2f89eb60d8acf2e Bug-Url: https://bugzilla.redhat.com/873595 Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/TryBackToAllSnapshotsOfVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java 2 files changed, 16 insertions(+), 12 deletions(-) Approvals: Maor Lipchuk: Looks good to me, approved Liron Aravot: Verified -- To view, visit http://gerrit.ovirt.org/9128 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6e8edc74bc34676f526dfd24d2f89eb60d8acf2e Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: cleanup ActivateDeactivateVmNicCommand
by lhornyak＠redhat.com 12 Nov '12

12 Nov '12

Laszlo Hornyak has submitted this change and it was merged. Change subject: engine: cleanup ActivateDeactivateVmNicCommand ...................................................................... engine: cleanup ActivateDeactivateVmNicCommand use getVmNetworkInterfaceDao rather than DbFacade.getinstance().getVmNetworkInterfaceDao() Change-Id: Iabbfa46b3cbd9d0b01a10648583bcf90d9042bba Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ActivateDeactivateVmNicCommand.java 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9126 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iabbfa46b3cbd9d0b01a10648583bcf90d9042bba Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Fixing NPE at RegisterVdsQuery
by mkublin＠redhat.com 12 Nov '12

12 Nov '12

Michael Kublin has submitted this change and it was merged. Change subject: engine: Fixing NPE at RegisterVdsQuery ...................................................................... engine: Fixing NPE at RegisterVdsQuery The following patch will fix a NPE which can occurred at RegisterVdsQuery during writting to logs Change-Id: I03199e5d16671d43b74e37dd28215bccfd4467e1 Signed-off-by: Michael Kublin <mkublin(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RegisterVdsQuery.java 1 file changed, 0 insertions(+), 4 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9177 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I03199e5d16671d43b74e37dd28215bccfd4467e1 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0