Details

    • Type: Improvement
    • Status: Open
    • Priority: Critical
    • Resolution: Unresolved
    • Affects Version/s: 2.4
    • Fix Version/s: 2.6
    • Component/s: Development
    • Security Level: Public (Public: Anyone can view)
    • Labels:
    • 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.

        Attachments

          Issue Links

            Activity

            gilesp Peter Giles (Inactive) created issue -
            gilesp Peter Giles (Inactive) made changes -
            Field Original Value New Value
            Link This issue relates to KULRICE-12596 [ KULRICE-12596 ]
            gilesp Peter Giles (Inactive) made changes -
            Link This issue relates to RICEQA-10 [ RICEQA-10 ]
            Hide
            gilesp Peter Giles (Inactive) added a comment - - edited

            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.

            Show
            gilesp Peter Giles (Inactive) added a comment - - edited 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.
            gilesp Peter Giles (Inactive) made changes -
            Summary Clean up KRAD datasource configuration Clean up datasource configuration
            kbtaylor Kristina Taylor (Inactive) made changes -
            Rank Ranked higher
            kbtaylor Kristina Taylor (Inactive) made changes -
            Issue Type Task [ 3 ] Improvement [ 4 ]
            kbtaylor Kristina Taylor (Inactive) made changes -
            Priority Blocker [ 1 ] Critical [ 2 ]
            kbtaylor Kristina Taylor (Inactive) made changes -
            Fix Version/s 2.6 [ 17820 ]
            Fix Version/s 2.5 [ 17044 ]
            kbtaylor Kristina Taylor (Inactive) made changes -
            Rank Ranked higher
            ewestfal Eric Westfall made changes -
            Labels Old

              People

              • Assignee:
                Unassigned
                Reporter:
                gilesp Peter Giles (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated: