[KULRICE-12814] Clean up datasource configuration Created: 10/Jun/14  Updated: 16/Jan/15

Status: Open
Project: Kuali Rice Development
Component/s: Development
Affects Version/s: 2.4
Fix Version/s: 2.6
Security Level: Public (Public: Anyone can view)

Type: Improvement Priority: Critical
Reporter: Peter Giles (Inactive) Assignee: Unassigned
Resolution: Unresolved Votes: 0
Labels: Old
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Relate
relates to KULRICE-13348 KRAD Client Sample App using standalo... Closed
relates to KULRICE-12596 Simplify Client Spring Bean configura... Open
KAI Review Status: Not Required
KTI Review Status: Not Required
Code Review Status: Not Required
Include in Release Notes?:
Yes

 Description   

There is some cleanup that needs to be done on our datasource configuration, especially with regards to KRAD.

We have three different jndi locations referenced on our project related to KRAD, and (as far as I can tell) there should only be one:

rice-middleware/impl/src/main/resources/META-INF/common-config-defaults.xml
<param name="kradServer.datasource.jndi.location" override="false"></param>
<param name="kradApplication.datasource.jndi.location" override="false"></param>
rice-framework/krad-web-framework/src/main/resources/org/kuali/rice/krad/config/KRADSpringBeans-data.xml
                             <value>kradClient.datasource</value>
                     </list>
...
                             <value>kradClient.datasource.jndi.location</value>
                     </list>

Additionally, it currently requires bean overrides to deploy our KRAD sample app as client (non-bundled), and we need to clean that up as well. Dan Seibert had to go through some hoops to get this set up in our test environments, and he shared some info on what he had to do:

override the coreConfigurer bean (which is in RiceServiceRegistrySpringBeans.xml) to

  • add a property: <property name="serverDataSource" ref="riceDataSource$ {connection.pool.impl}

    " />

  • modify the dataSource property to reference a client datasource bean (should perhaps be called clientDataSource)

override the kradApplicationDataSource alias from _KradSampleAppJpaSpringBeans.xml to reference clientDataSource instead

  • For bundled config.xml, define both a client and server datasource pointing at the same DB.
  • For client app config.xml, define both datasources pointing at separate DBs
  • need to add rice.server.datasource.username & rice.server.datasource.password as well

We need to clean that up so that we can deploy the sample app in different vanilla configurations without resorting to bean overrides.

We will definitely need to document changes we make on our impacting changes wiki page so our customers have a heads up and can figure out how to clean their configurations up too.



 Comments   
Comment by Peter Giles (Inactive) [ 10/Jun/14 ]

I set this to blocker because it is preventing automated deployments of our sample app in non-bundled mode (there has to be a better way to say that!) for our QA environments.

Generated at Mon Sep 28 15:32:57 CDT 2020 using JIRA 7.0.11#70121-sha1:19d24976997c1d95f06f3e327e087be0b71f28d4.