[Engine-patches] Change in ovirt-engine[master]: restapi: Remove spring framework

2012-07-02 Thread oourfali
Oved Ourfali has submitted this change and it was merged. Change subject: restapi: Remove spring framework .. restapi: Remove spring framework Remove the dependency on the spring framework. Change-Id: I3f27d340261e4b5c4b6b7f27

[Engine-patches] Change in ovirt-engine[master]: restapi: Remove spring framework

2012-07-02 Thread oourfali
Oved Ourfali has posted comments on this change. Change subject: restapi: Remove spring framework .. Patch Set 8: Verified; Looks good to me, approved rebased. based on previous acks. -- To view, visit http://gerrit.ovirt.org/

[Engine-patches] Change in ovirt-engine[master]: restapi: Remove spring framework

2012-07-02 Thread eedri
Eyal Edri has posted comments on this change. Change subject: restapi: Remove spring framework .. Patch Set 7: Verified -- To view, visit http://gerrit.ovirt.org/1390 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerr

[Engine-patches] Change in ovirt-engine[master]: restapi: Remove spring framework

2012-07-01 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: restapi: Remove spring framework .. Patch Set 7: Looks good to me, approved +2 for api -- To view, visit http://gerrit.ovirt.org/1390 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: restapi: Remove spring framework

2012-06-26 Thread oourfali
Oved Ourfali has posted comments on this change. Change subject: restapi: Remove spring framework .. Patch Set 6: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/1390 To unsubscribe, vi

[Engine-patches] Change in ovirt-engine[master]: restapi: Remove spring framework

2012-06-26 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: restapi: Remove spring framework .. Patch Set 6: Verified -- To view, visit http://gerrit.ovirt.org/1390 To unsubscribe, visit http://gerrit.ovirt.org/settings

[Engine-patches] Change in ovirt-engine[master]: restapi: Remove Spring Framework

2012-05-31 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: restapi: Remove Spring Framework .. Patch Set 5: Verified Verified that building a new set of RPMs the result can be installed and the new disks resource works

[Engine-patches] Change in ovirt-engine[master]: restapi: Remove Spring Framework

2012-05-31 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: restapi: Remove Spring Framework .. Patch Set 5: > it should be rebased (manually) as was added new root coll. > (backenddisksresource) + i would like QE to ma

[Engine-patches] Change in ovirt-engine[master]: restapi: Remove Spring Framework

2012-05-31 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: restapi: Remove Spring Framework .. Patch Set 4: I would prefer that you didn't submit this it should be rebased (manually) as was added new root coll. (bac

[Engine-patches] Change in ovirt-engine[master]: restapi: Remove Spring Framework

2012-03-22 Thread oourfali
Oved Ourfali has posted comments on this change. Change subject: restapi: Remove Spring Framework .. Patch Set 2: Looks good to me, but someone else must approve Also, I opened a pre-integration ticket on testing my patches for

[Engine-patches] Change in ovirt-engine[master]: restapi: Remove Spring Framework

2012-03-22 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: restapi: Remove Spring Framework .. Patch Set 2: Good news: version 7.1.1 of JBoss AS includes version 2.3.2 of resteasy, which fixes our issues. -- To view,

[Engine-patches] Change in ovirt-engine[master]: restapi: Remove Spring Framework

2012-02-29 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: restapi: Remove Spring Framework .. Patch Set 2: Updated to use RESTEasy 2.3.2.Final which fixes the problem with the "/" URL. Bad news is that as we are depen

[Engine-patches] Change in ovirt-engine[master]: restapi: Remove Spring Framework (#NA)

2012-02-12 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: restapi: Remove Spring Framework (#NA) .. Patch Set 1: (2 inline comments) File backend/manager/modules/

[Engine-patches] Change in ovirt-engine[master]: restapi: Remove Spring Framework (#NA)

2012-02-10 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: restapi: Remove Spring Framework (#NA) .. Patch Set 1: The bug with the "/api" URL has been fixed upstream in RESTEasy: https://issues.jboss.org/browse/RESTEAS

[Engine-patches] Change in ovirt-engine[master]: restapi: Remove Spring Framework (#NA)

2012-01-31 Thread juan . hernandez
Juan Hernandez has uploaded a new change for review. Change subject: restapi: Remove Spring Framework (#NA) .. restapi: Remove Spring Framework (#NA) No bug. Remove the dependency on the Spring Framework. Change-Id: I3f27d3402