Conversation
stevenwinship
left a comment
There was a problem hiding this comment.
I had to run the following to create the base image before building and starting the docker containers
mvn -Pct -f modules/container-base install
|
FWIW: Payara 6.2025.10 also has significant improvements, so I updated the PR. In doing that, it looks like some change caused our guestbook estimate for downloads stored procedure to fail. I've switched how that works, now using a native query of that procedure which appears to work fine. (I don't know exactly why the old method started giving exceptions, but this Payara does update the EclipseLink version used.) |
|
This now passes all tests on Payara 6.2025.10, ready any re-review (only the mime file changes are new since the earlier review) and QA. https://jenkins.dataverse.org/job/IQSS-Dataverse-Payara6/94/ |
|
@qqmyers I'm merging this now (got distracted by some unrelated time sinks during the day). |
What this PR does / why we need it: Updates to the newest Payara that has bug fixes/security updates
Which issue(s) this PR closes:
Special notes for your reviewer: FWIW: With this Payara version's update of EclipseLink and other libraries, our guestbook download estimate query stopped working and there were many errors related to the mime.types file needed to be renamed (see payara/Payara#7741).
The PR was reviewed prior to the minor addition of the the jakarata.mime.types file.
The latest Jenkins run is at https://jenkins.dataverse.org/job/IQSS-Dataverse-Payara6/93/console
Suggestions on how to test this: Follow the docs to setup a new machine and to upgrade an existing machine. Note that you cannot run Dataverse 6.8 (or anything prior to this PR) on Payara 6.2025.10.
Does this PR introduce a user interface change? If mockups are available, please link/include them here:
Is there a release notes update needed for this change?:
Additional documentation: