January 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: Update the commons-lang 2.6
by juan.hernandez＠redhat.com 10 Jan '13

10 Jan '13

Juan Hernandez has submitted this change and it was merged. Change subject: core: Update the commons-lang 2.6 ...................................................................... core: Update the commons-lang 2.6 This is the version that we are using in deployments as it is included in Fedora, just need to sync the POM to use it as well. Change-Id: I110a7d1f8fb7178d0ad8a0900fbc73af4a9bac69 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M pom.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9268 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I110a7d1f8fb7178d0ad8a0900fbc73af4a9bac69 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Asaf Shakarchi <asaf(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: Remove overriden JBoss modules
by juan.hernandez＠redhat.com 10 Jan '13

10 Jan '13

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: Remove overriden JBoss modules ...................................................................... packaging: Remove overriden JBoss modules Remove some of the JBoss modules that we replaced in the past. They are no longer needed because the latest version of the JBoss package already includes them. Change-Id: I6c4ece73cb8400d6e51d70cb8ae34a309675bfbc Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- D deployment/modules/com/sun/xml/bind/main/module.xml D deployment/modules/javax/ws/rs/api/main/module.xml D deployment/modules/org/apache/commons/codec/main/module.xml D deployment/modules/org/apache/httpcomponents/main/module.xml D deployment/modules/org/apache/james/mime4j/main/module.xml D deployment/modules/org/codehaus/jackson/jackson-core-asl/main/module.xml D deployment/modules/org/codehaus/jackson/jackson-jaxrs/main/module.xml D deployment/modules/org/codehaus/jackson/jackson-mapper-asl/main/module.xml D deployment/modules/org/codehaus/jackson/jackson-xc/main/module.xml D deployment/modules/org/codehaus/jettison/main/module.xml D deployment/modules/org/hibernate/validator/main/module.xml D deployment/modules/org/jboss/resteasy/resteasy-atom-provider/main/module.xml D deployment/modules/org/jboss/resteasy/resteasy-cdi/main/module.xml D deployment/modules/org/jboss/resteasy/resteasy-jackson-provider/main/module.xml D deployment/modules/org/jboss/resteasy/resteasy-jaxb-provider/main/module.xml D deployment/modules/org/jboss/resteasy/resteasy-jaxrs/main/module.xml D deployment/modules/org/jboss/resteasy/resteasy-jettison-provider/main/module.xml D deployment/modules/org/jboss/resteasy/resteasy-jsapi/main/module.xml D deployment/modules/org/jboss/resteasy/resteasy-multipart-provider/main/module.xml D deployment/modules/org/jboss/resteasy/resteasy-yaml-provider/main/module.xml D deployment/modules/org/scannotation/scannotation/main/module.xml D deployment/modules/org/yaml/snakeyaml/main/module.xml M packaging/fedora/spec/ovirt-engine.spec.in 23 files changed, 1 insertion(+), 952 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9402 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6c4ece73cb8400d6e51d70cb8ae34a309675bfbc Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0

Change in ovirt-engine[master]: tools: Remove obsolete configuration parameters
by juan.hernandez＠redhat.com 10 Jan '13

10 Jan '13

Juan Hernandez has submitted this change and it was merged. Change subject: tools: Remove obsolete configuration parameters ...................................................................... tools: Remove obsolete configuration parameters The parameters in the engine-config.conf file that point to the location of the main application server configuration file are no longer needed. Change-Id: I1ae738c43dc78c6d7125529aeb87039eb74c794c Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/tools/engine-config/src/main/resources/engine-config.conf 1 file changed, 0 insertions(+), 6 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8782 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1ae738c43dc78c6d7125529aeb87039eb74c794c Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Sharad Mishra <snmishra(a)linux.vnet.ibm.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: avoid NPE on UpdateVM template command
by mlipchuk＠redhat.com 10 Jan '13

10 Jan '13

Maor Lipchuk has submitted this change and it was merged. Change subject: core: avoid NPE on UpdateVM template command ...................................................................... core: avoid NPE on UpdateVM template command When trying to update a template which was deleted, an NPE can occur whent trying to load the template disks from DB. Change-Id: I6788f10a6faef632f7ef13f39a85ede9f4fdb696 Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmTemplateCommand.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/10545 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6788f10a6faef632f7ef13f39a85ede9f4fdb696 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Alissa Bonas <abonas(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: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: avoid unneeded db calls on update of BlankTemplate
by mlipchuk＠redhat.com 10 Jan '13

10 Jan '13

Maor Lipchuk has submitted this change and it was merged. Change subject: core: avoid unneeded db calls on update of BlankTemplate ...................................................................... core: avoid unneeded db calls on update of BlankTemplate When trying to edit a blank template, there's no need to load the template from the DB before failing the operation. Change-Id: Ia66a00303c19d9fe77fbd57021227f08ad5d160b Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmTemplateCommand.java 1 file changed, 4 insertions(+), 3 deletions(-) Approvals: Maor Lipchuk: Looks good to me, approved Liron Aravot: Verified -- To view, visit http://gerrit.ovirt.org/10544 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia66a00303c19d9fe77fbd57021227f08ad5d160b Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(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: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: disks subtab - enable template disk removal
by derez＠redhat.com 09 Jan '13

09 Jan '13

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: disks subtab - enable template disk removal ...................................................................... webadmin: disks subtab - enable template disk removal Enable template disk removal in disks sub-tab (under storage main-tab). Change-Id: Ieff2f7a999cc1baa34ae85c34a51c83e19600bed Bug-Url: https://bugzilla.redhat.com/892714 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageDiskListModel.java 1 file changed, 2 insertions(+), 4 deletions(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10817 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieff2f7a999cc1baa34ae85c34a51c83e19600bed Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: modified title of index and 404 HTML pages
by derez＠redhat.com 09 Jan '13

09 Jan '13

Daniel Erez has submitted this change and it was merged. Change subject: engine: modified title of index and 404 HTML pages ...................................................................... engine: modified title of index and 404 HTML pages Modified title: 'OVirt' -> 'oVirt' Change-Id: I7d22ddf9d138142f23bca3fd92c07d9178f8cc77 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/modules/root/src/main/webapp/404.html M backend/manager/modules/root/src/main/webapp/index.html M frontend/webadmin/modules/userportal-gwtp/src/main/webapp/404.html M frontend/webadmin/modules/webadmin/src/main/webapp/404.html 4 files changed, 4 insertions(+), 4 deletions(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10829 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7d22ddf9d138142f23bca3fd92c07d9178f8cc77 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: TarTest - assume Unix OS
by derez＠redhat.com 09 Jan '13

09 Jan '13

Daniel Erez has submitted this change and it was merged. Change subject: core: TarTest - assume Unix OS ...................................................................... core: TarTest - assume Unix OS TarTest requires tar utility. Therefore, ignoring test for non-Unix machines. Change-Id: I1c80cb652f4d3e633295d9bd88bc05d806c1d34f Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/archivers/tar/TarTest.java 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10739 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1c80cb652f4d3e633295d9bd88bc05d806c1d34f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: fix to dao test
by ofrenkel＠redhat.com 09 Jan '13

09 Jan '13

Omer Frenkel has submitted this change and it was merged. Change subject: core: fix to dao test ...................................................................... core: fix to dao test this patch fixes VmAndTemplatesGenerationsDaoTest testDeleteByOvfGenerations() Change-Id: I70c1ca64762de3b5827cb1bfff34b4dd35d4edaa Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmAndTemplatesGenerationsDaoTest.java 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Omer Frenkel: Looks good to me, approved Liron Aravot: Verified -- To view, visit http://gerrit.ovirt.org/10822 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I70c1ca64762de3b5827cb1bfff34b4dd35d4edaa Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: fix for BackendCdRomResourceTest
by lhornyak＠redhat.com 09 Jan '13

09 Jan '13

Laszlo Hornyak has submitted this change and it was merged. Change subject: core: fix for BackendCdRomResourceTest ...................................................................... core: fix for BackendCdRomResourceTest fixed property name from iso_path to IsoPath Change-Id: Id89d3aaeb2092fcea583825accf0e58dcb1a2b9b Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendCdRomResourceTest.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10803 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id89d3aaeb2092fcea583825accf0e58dcb1a2b9b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0