IQSS/11126 payara6.2025.1

[qqmyers]

What this PR does / why we need it: Updates to Payara 6.2025.1, fixing a few places where JSF now appears to send a null rather than an empty string, causing some new NPEs.

Which issue(s) this PR closes:

• Closes Upgrading to Payara 6.2024.12 #11126

Special notes for your reviewer: FWIW: It looks like just copying the domain across to the new version works so we may only want to show 'Option 1' form the release notes. I originally did 'Option2' - just copying the parts of domain.xml and other files to the new domain which has some advantages (easier to diff against the latest payara domain.xml, gets new cacerts file, makes it possible to track payara domain.xml changes in github (against /conf/payara/domain.xml), etc.) but it's definitely more work/more confusing.

With further testing at QDR, I've noticed a steady/overwhelming stream of errors of the form JSF1103: The metadata facet must be a direct child of the view in viewId /dataverse.xhtml]]. I did not notice any change in UI behavior though. Digging in a bit, it appears that any page that uses <ui:composition template="/dataverse_template.xhtml"> was including the f:metadata farther down in the tree. I've made changes to move them to being a direct child of the view which appears to get rid of the log messages, again w/o any impact on functionality that I've seen. I haven't been able to confirm, but my guess is that while the error is now getting flagged, the f:metadata was still be interpreted correctly.

Suggestions on how to test this: Check new installs, docker, upgrade instructions - basically regression testing and testing instructions, there shouldn't be any functional changes. Could confirm that the current v6.5 war does not work with Payara 6.2025.1, but I think that's well enough known.

Testing should also involve grepping for JSF1103 in the logs to assure that there are no more instances (e.g. .xhtml files I missed.)

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?: included

Additional documentation: references to 6.2024.6 changed to 6.2025.1 in docs

[qqmyers]

@poikilotherm - any thoughts on the MailSessionProducerIT test failure? I don't see how the code changes could affect this but perhaps something has changed in Payara/its config? (I looked at another PR where this passes and I think the hostname is localhost there as well, so I'm not sure what might have changed.)

[qqmyers]

FWIW: The current test failures in Jenkins are from this PR running against Payara 6.2023.8 - not sure if that is also an issue for the Maven Tests.

[donsizemore]

@qqmyers I can set up a separate Jenkins job to test this pretty easily. I'll try to do that today.

[donsizemore]

integration tests succeeded: Tests run: 286, Failures: 0, Errors: 0, Skipped: 13

[qqmyers]

If this option is kept, the cd should be to /usr/local/glassfish/domains/domain1/config
and curl should have the -L -O options