Bug 793235 (JBEPP-320) - New mechanism for passing configuration directory location
Summary: New mechanism for passing configuration directory location
Keywords:
Status: CLOSED NEXTRELEASE
Alias: JBEPP-320
Product: JBoss Enterprise Portal Platform 5
Classification: JBoss
Component: unspecified
Version: 5.0.0.ER04
Hardware: Unspecified
OS: Unspecified
urgent
urgent
Target Milestone: ---
: 5.0.0.CR01
Assignee: Martin Podolinsky
QA Contact:
URL: http://jira.jboss.org/jira/browse/JBE...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-04-21 09:32 UTC by Dominik Pospisil
Modified: 2010-04-22 13:29 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
EPP5.CR1-snapshot-100414
Last Closed: 2010-04-22 13:29:10 UTC
Type: Feature Request


Attachments (Terms of Use)
properties-service.xml (1.70 KB, text/xml)
2010-04-21 13:46 UTC, Marko Strukelj
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker JBEPP-320 0 None Closed New mechanism for passing configuration directory location 2012-06-07 21:34:58 UTC

Description Dominik Pospisil 2010-04-21 09:32:17 UTC
Date of First Response: 2010-04-21 08:35:18
project_key: JBEPP

The Gatein portal configuration directory location is passed to EPP through system property exo.conf.dir.name. This breaks compatibility with current tooling (JBT 3.1, JBDS 3.0.0.GA) as both JBT and JBDS do not read run.conf script to start the runtime. It would be nice if the requirement to pass the property to Gatein is removed in EPP.

Comment 1 Dominik Pospisil 2010-04-21 09:33:08 UTC
Link: Added: This issue is related to JBIDE-6202


Comment 2 Marko Strukelj 2010-04-21 12:35:18 UTC
The solution to this is very simple really.

We can just put the required system properties into JBOSS_HOME/server/default/deploy/properties-service.xml

This will set System properties before Kernel's RootContainer is initialized, so effectively there is no difference for us between this and the current configuration approach via -D command line option.

A little downside is that properties-service will override any -D option for the same key - a little bit non-intuitive.


I'll see if I can write a simple alternative for our purposes that would avoid overriding the already set properties, and put it all together in a non-obtrusive way.


Comment 3 Marko Strukelj 2010-04-21 13:46:25 UTC
Attached  properties-services.xml

This should do the trick.

Comment 4 Marko Strukelj 2010-04-21 13:46:25 UTC
Attachment: Added: properties-service.xml


Comment 5 Martin Podolinsky 2010-04-22 13:29:10 UTC
Issue fixed by altering the packaging script as proposed.


Note You need to log in before you can comment on or make changes to this bug.