Change in ovirt-engine[master]: restapi: Deploy using modules

juan.hernandez at redhat.com juan.hernandez at redhat.com
Fri Jul 4 09:07:19 UTC 2014


Juan Hernandez has submitted this change and it was merged.

Change subject: restapi: Deploy using modules
......................................................................


restapi: Deploy using modules

Currently the RESTAPI application is deployed as a nested .war file
inside the .ear of the engine, actually as two .war files, one for /api,
and another for /ovirt-engine/api. This double deployment causes class
cast exceptions because classes are loaded twice. To avoid this issues
this patch changes the RESTAPI application so that it is deployed
outside of the .ear and users JBoss modules for all its components, so
that no class will be loaded twice.

Change-Id: Ie4079277f60e5521d9b61379fa78713a5c4ccc10
Bug-Url: https://bugzilla.redhat.com/1113485
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
---
M Makefile
M backend/manager/modules/branding/pom.xml
A backend/manager/modules/branding/src/main/modules/org/ovirt/engine/core/branding/main/module.xml
M backend/manager/modules/restapi/interface/common/jaxrs/pom.xml
M backend/manager/modules/restapi/interface/common/jaxrs/src/main/java/org/ovirt/engine/api/common/util/PackageExplorer.java
A backend/manager/modules/restapi/interface/common/jaxrs/src/main/modules/org/ovirt/engine/api/interface-common-jaxrs/main/module.xml
M backend/manager/modules/restapi/interface/definition/pom.xml
D backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/utils/FileUtils.java
A backend/manager/modules/restapi/interface/definition/src/main/modules/org/ovirt/engine/api/restapi-definition/main/module.xml
M backend/manager/modules/restapi/jaxrs/pom.xml
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/validation/ValidatorLocator.java
A backend/manager/modules/restapi/jaxrs/src/main/modules/org/ovirt/engine/api/restapi-jaxrs/main/module.xml
M backend/manager/modules/restapi/types/pom.xml
M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/MappingLocator.java
A backend/manager/modules/restapi/types/src/main/modules/org/ovirt/engine/api/restapi-types/main/module.xml
M backend/manager/modules/restapi/webapp/pom.xml
D backend/manager/modules/restapi/webapp/src/main/resources/META-INF/MANIFEST.MF
A backend/manager/modules/restapi/webapp/src/main/webapp/WEB-INF/jboss-deployment-structure.xml
A backend/manager/modules/restapi/webapp/src/main/webapp/WEB-INF/jboss-web.xml
M backend/manager/modules/restapi/webapp/src/main/webapp/WEB-INF/web.xml
M ear/pom.xml
M ear/src/main/resources/META-INF/MANIFEST.MF
M ovirt-engine.spec.in
M packaging/services/ovirt-engine/ovirt-engine.conf.in
25 files changed, 524 insertions(+), 239 deletions(-)

Approvals:
  Juan Hernandez: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/29474
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie4079277f60e5521d9b61379fa78713a5c4ccc10
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list