June 2012 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: restapi: Handle PermGen leak for Maven 3
by lhornyak＠redhat.com 13 Jun '12

13 Jun '12

Laszlo Hornyak has submitted this change and it was merged. Change subject: restapi: Handle PermGen leak for Maven 3 ...................................................................... restapi: Handle PermGen leak for Maven 3 This package uses PowerMock (extensively) in it's tests, which unfortunately leak PermGen memory. This went unnoticed in Maven 2, but in Maven 3, which has larger memory requirements, causes the build to consistantly fail on NicMapperTest. In order to avoid this problem the tests were forked in a similar way as in the bll projcet. Change-Id: I1f6e07281ed4b1653aae7e4c579ec6d587c6987f Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/restapi/types/pom.xml 1 file changed, 17 insertions(+), 0 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5255 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1f6e07281ed4b1653aae7e4c579ec6d587c6987f 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: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-engine[master]: Update API test with searchable disks
by oliel＠redhat.com 13 Jun '12

13 Jun '12

Ori Liel has submitted this change and it was merged. Change subject: Update API test with searchable disks ...................................................................... Update API test with searchable disks Change-Id: Ifdbedea4427aec11116cc542f1fd1fbbaab2fb57 Signed-off-by: Ori Liel <oliel(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendApiResourceTest.java 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Michael Pasternak: Looks good to me, approved Ori Liel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5303 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifdbedea4427aec11116cc542f1fd1fbbaab2fb57 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ori Liel <oliel(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]: restapi: Redefine Host Cores And Sockets
by oliel＠redhat.com 13 Jun '12

13 Jun '12

Ori Liel has submitted this change and it was merged. Change subject: restapi: Redefine Host Cores And Sockets ...................................................................... restapi: Redefine Host Cores And Sockets Change-Id: Icdd4ca443afa0358b9c46cb7754ce64901929912 Signed-off-by: Ori Liel <oliel(a)redhat.com> --- M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostMapper.java M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/HostMapperTest.java 2 files changed, 7 insertions(+), 5 deletions(-) Approvals: Ori Liel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5239 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icdd4ca443afa0358b9c46cb7754ce64901929912 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ori Liel <oliel(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]: restapi: Add Search Capabilities To Disks
by oliel＠redhat.com 13 Jun '12

13 Jun '12

Ori Liel has submitted this change and it was merged. Change subject: restapi: Add Search Capabilities To Disks ...................................................................... restapi: Add Search Capabilities To Disks Change-Id: Ief1e566756c454fb2947f5dff0a54f16c36a65ec Signed-off-by: Ori Liel <oliel(a)redhat.com> --- M backend/manager/modules/restapi/interface/common/jaxrs/src/main/java/org/ovirt/engine/api/common/util/QueryHelper.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/BackendApiResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendDisksResource.java 4 files changed, 7 insertions(+), 4 deletions(-) Approvals: Ori Liel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5291 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ief1e566756c454fb2947f5dff0a54f16c36a65ec Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ori Liel <oliel(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]: restapi: Expose Current Time
by oliel＠redhat.com 13 Jun '12

13 Jun '12

Ori Liel has submitted this change and it was merged. Change subject: restapi: Expose Current Time ...................................................................... restapi: Expose Current Time Change-Id: I2be1681cb6cac28b3f624a700cb33a5f02b9a5c6 Signed-off-by: Ori Liel <oliel(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendApiResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendApiResourceTest.java 3 files changed, 5 insertions(+), 0 deletions(-) Approvals: Ori Liel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5242 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2be1681cb6cac28b3f624a700cb33a5f02b9a5c6 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ori Liel <oliel(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]: restapi: Fix RSDL Generics Issue
by oliel＠redhat.com 13 Jun '12

13 Jun '12

Ori Liel has submitted this change and it was merged. Change subject: restapi: Fix RSDL Generics Issue ...................................................................... restapi: Fix RSDL Generics Issue Change-Id: Icd97352f2a964198e1ad09a613cc9d1e10e0f614 Signed-off-by: Ori Liel <oliel(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/rsdl/RsdlBuilder.java 1 file changed, 31 insertions(+), 10 deletions(-) Approvals: Ori Liel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5287 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icd97352f2a964198e1ad09a613cc9d1e10e0f614 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ori Liel <oliel(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]: Fix warning for StorageHelperDirector
by lhornyak＠redhat.com 12 Jun '12

12 Jun '12

Laszlo Hornyak has submitted this change and it was merged. Change subject: Fix warning for StorageHelperDirector ...................................................................... Fix warning for StorageHelperDirector Change-Id: Iab0652d83b6fffa28fd4bc37a84ab616647cf813 Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/StorageHelperDirector.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/4681 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iab0652d83b6fffa28fd4bc37a84ab616647cf813 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi <smizrahi(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: fix ValidationResult
by lhornyak＠redhat.com 12 Jun '12

12 Jun '12

Laszlo Hornyak has submitted this change and it was merged. Change subject: core: fix ValidationResult ...................................................................... core: fix ValidationResult fixes constructor in ValidationResult Change-Id: I2ed1dea33bbe0786ec278d4d166e41d5057ad246 Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ValidationResult.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5293 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2ed1dea33bbe0786ec278d4d166e41d5057ad246 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com>

1 0

Change in ovirt-engine[master]: userportal: Edit Console Dialog fixes
by vszocs＠redhat.com 12 Jun '12

12 Jun '12

Vojtech Szocs has submitted this change and it was merged. Change subject: userportal: Edit Console Dialog fixes ...................................................................... userportal: Edit Console Dialog fixes Includes: - the dialog now has a title "Console Options" - the VM name is not hardcoded but displays the VM name correctly - the "Pass Ctrl-Alt-Del to virtual machine" is on one line Change-Id: Ide534fae54fab4029c4c90ec02b3d4b454ad0f5d Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com> --- M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/section/main/presenter/popup/console/ConsolePopupPresenterWidget.java M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/section/main/view/popup/console/ConsolePopupView.java M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/section/main/view/popup/console/ConsolePopupView.ui.xml 3 files changed, 38 insertions(+), 12 deletions(-) Approvals: Vojtech Szocs: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/4891 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ide534fae54fab4029c4c90ec02b3d4b454ad0f5d Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: hiding RDMA in create volume dialog
by gchaplik＠redhat.com 12 Jun '12

12 Jun '12

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: hiding RDMA in create volume dialog ...................................................................... webadmin: hiding RDMA in create volume dialog The RDMA checkbox in volume dialog is hidden and set to false as we are not supporting it currently. TCP is selected by default and disabled. Labels in the create volume dialog are localized. Change-Id: I8ce12ac0c0df92a5044e2e794f8f7c175612d0e3 Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/VolumePopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/VolumePopupView.ui.xml 4 files changed, 35 insertions(+), 5 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5286 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8ce12ac0c0df92a5044e2e794f8f7c175612d0e3 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com>

1 0