Skip to content
This repository has been archived by the owner on Jul 27, 2023. It is now read-only.

java-microprofile should support JEE JCA connection pools #539

Closed
nastacio opened this issue Dec 4, 2019 · 5 comments
Closed

java-microprofile should support JEE JCA connection pools #539

nastacio opened this issue Dec 4, 2019 · 5 comments
Assignees
Labels
stack/java-openliberty Issues related to java-openliberty stack

Comments

@nastacio
Copy link
Contributor

nastacio commented Dec 4, 2019

Is your feature request related to a problem? Please describe.
Currently, there is no support mechanism for placing JDBC/JMS/J2C drivers into the container image, which means one cannot declare connection pools in server.xml because they need to access the non-existent driver files during runtime.

Describe the solution you'd like
A supported mechanism for placing these types of JAR files in the container image in a location accessible via server.xml, e.g.:

    <library id="db2-library">
        <fileset 
            id="db2Fileset"
            dir="${server.config.dir}/resources"
            includes="jcc*.jar"/>
    </library>

Describe alternatives you've considered
There is a workaround based on using the Maven Resource Plugin to place the driver files in a strategic location of the build tree, from where they get picked up during appsody build. Note that this strategic location is gleaned from inside-knowledge of the current stack Dockerfile, currently: ${basedir}/target/liberty/wlp/usr/servers/defaultServer/resources/

Additional context
Add any other context or screenshots about the feature request here.

#521

@scottkurz scottkurz added the stack/java-microprofile Issues related to java-microprofile stack label Dec 5, 2019
@scottkurz
Copy link
Contributor

Assigned @ajm01 ... I think the solution here may ultimately involve copying the JDBC driver artifact into target/liberty/wlp/usr/servers/defaultServer/resources/ , but we should have a pattern (or even a template) to make it more obvious how to do this, so it doesn't feel like each person needs to come up with their own hack/solution.

@scottkurz scottkurz added the stack/java-openliberty Issues related to java-openliberty stack label Jan 13, 2020
@scottkurz
Copy link
Contributor

Though this issue hasn't been solved in the java-microprofile stack, relabeling to the java-openliberty stack, which is where we will prioritize a solution.

@scottkurz scottkurz removed the stack/java-microprofile Issues related to java-microprofile stack label Jan 13, 2020
@scottkurz
Copy link
Contributor

See discussion in LMP issue 705.

@scottkurz
Copy link
Contributor

@nastacio would you say we can close this now ?

@nastacio
Copy link
Contributor Author

nastacio commented Mar 3, 2020

Yes, I also wrote an issue for myself to update
https://developer.ibm.com/tutorials/connecting-appsody-java-microprofile-applications-to-jdbc-data-sources/
with the new stack

thanks @scottkurz !

@nastacio nastacio closed this as completed Mar 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
stack/java-openliberty Issues related to java-openliberty stack
Projects
None yet
Development

No branches or pull requests

3 participants