Change in ovirt-engine[master]: packaging: Use temporary modules directory

juan.hernandez at redhat.com juan.hernandez at redhat.com
Wed Jan 9 13:40:32 UTC 2013


Juan Hernandez has submitted this change and it was merged.

Change subject: packaging: Use temporary modules directory
......................................................................


packaging: Use temporary modules directory

The application server creates .jar.index files in the modules directory
in order to speed up class scanning and class loading. When the
application server is updated those files are not refreshed unless the
application server is started with an user that has permission to write
to the /usr/share/jbossas/modules directory. The engine doesn't have
those permissions, so it can end up using out of date indexes, and this
can generate class loading this problems. To address that issue this
patch changes the engine start script so that it creates a temporary
copy of the modules directory, containing links to the .xml and .jar
files and excluding the .jar.index files.

Change-Id: I4f27390a232df42b477dc0e9abe7909e10d9abb1
Bug-Url: https://bugzilla.redhat.com/889217
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
---
M packaging/fedora/engine-service.py
1 file changed, 25 insertions(+), 3 deletions(-)

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4f27390a232df42b477dc0e9abe7909e10d9abb1
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Moran Goldboim <mgoldboi at redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib at redhat.com>



More information about the Engine-commits mailing list