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(a)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(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby(a)redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server