I agree that starting the existing jsr88-* configs is a better solution than adding gbeans to the deployer configs. There might possibly be classloading problems from duplicate copies of e.g. geronimo-connector-builder.jar, but if that happens I think the better solution is to move the jsr88 code into separate jars, something I believe Aaron and I have been contemplating for years.

I'm glad to know that there is already a LocalDeploymentManager.

thanks
david jencks

On Mar 6, 2007, at 9:11 AM, Gianny Damour wrote:

Hello Anita,

I had a quick look to GERONIMO-2916 and, as per David J. comment, it seems to me that if you simply start the pointed out modules this bug will be fixed: DatabasePoolPortlet gets a LocalDeploymentManager instance, which knows about all the running ModuleConfigurer GBean implementations. If org.apache.geronimo.configs/jsr88-rar-configurer//car is started, then you will get a RARConfigurer running within the server. Then when DatabasePoolPortlet obtains its LocalDeploymentManager, this latter does know about RARConfigurer.

Thanks,
Gianny


On 07/03/2007, at 12:18 AM, Anita Kulshreshtha wrote:

Minor (? :)) Clarification....
   David, I meant to write connector-deployer config not
system-database to add the GBean to. Will that change your answer? I
think having individual GBeans in the appropriate deployer config will
work well for minimal and framework servers.

Thanks
Anita

--- Anita Kulshreshtha <[EMAIL PROTECTED]> wrote:


--- David Jencks <[EMAIL PROTECTED]> wrote:


On Mar 5, 2007, at 6:31 PM, Anita Kulshreshtha wrote:

   I need to add RARConfigurer GBean to system-database config to
fix
https://issues.apache.org/jira/browse/GERONIMO-2916 (see excerpts
below)
   Is this the correct place to add it?

No.

Instead of adding gbeans to anything, I think you want to start all

the jsr88-*configurer modules gianny recently added.   See
assemblies/

geronimo-jetty6-jee5/src/main/var/config/jsr88-configurer-config.xml

which contains

   <module
name="org.apache.geronimo.configs/jsr88-cli/${version}/car"/>
   <module name="org.apache.geronimo.configs/jsr88-jar-configurer/$

{version}/car"/>
   <module name="org.apache.geronimo.configs/jsr88-rar-configurer/$

{version}/car"/>
   <module name="org.apache.geronimo.configs/jsr88-war-configurer/$

{version}/car"/>
   <module name="org.apache.geronimo.configs/jsr88-ear-configurer/$

{version}/car"/>


I don't think you want to  start the jsr88-cli in the server.  If
there isn't already an appropriate gbean for the DeploymentManager

we'd need a gbean like

     <gbean name="ModuleConfigurerRegistry"


class="org.apache.geronimo.deployment.plugin.jmx.RemoteDeploymentMana ger

">
         <references name="ModuleConfigurers">
             <pattern>
                 <name>ClientConfigurer</name>
             </pattern>
             <pattern>
                 <name>EARConfigurer</name>
             </pattern>
             <pattern>
                 <name>RARConfigurer</name>
             </pattern>
             <pattern>
                 <name>WARConfigurer</name>
             </pattern>
         </references>
     </gbean>

somewhere appropriate but I don't think it should be the
jmxRemoteDeploymentManager.  (but I'm not sure)  It does need to be

able to accept the *Configurers registering with it.

     I found WARConfigurer GBean in tomcat6-deployer config. I did
not
find any EarConfigurer. I am still mesmerized by this stuff... The
ModuleConfigurerRegistry is a good choice. Where should this GBean
go?

Thanks
Anita



thanks
david jencks


Thanks
Anita

........................................................
5:07:55,562 ERROR [DatabasePoolPortlet] Unable to save connection
pool
javax.enterprise.deploy.spi.exceptions.InvalidModuleException: No
configurer for module type: rar registered
at



org.apache.geronimo.deployment.plugin.jmx.JMXDeploymentManager.create C

onfiguration(JMXDeploymentManager.java:302)







_____________________________________________________________________ _

______________
Don't get soaked.  Take a quick peak at the forecast
with the Yahoo! Search weather shortcut.
http://tools.search.yahoo.com/shortcuts/#loc_weather







_____________________________________________________________________ _______________
Do you Yahoo!?
Everyone is raving about the all-new Yahoo! Mail beta.
http://new.mail.yahoo.com





_____________________________________________________________________ _______________
Never miss an email again!
Yahoo! Toolbar alerts you the instant new Mail arrives.
http://tools.search.yahoo.com/toolbar/features/mail/


Reply via email to