Skip to content

DRYD-2090: Updates to Nuxeo Deployment - #539

Merged
mikejritter merged 5 commits into
collectionspace:developfrom
mikejritter:task/dryd-2090-nuxeo-build
Jun 18, 2026
Merged

DRYD-2090: Updates to Nuxeo Deployment#539
mikejritter merged 5 commits into
collectionspace:developfrom
mikejritter:task/dryd-2090-nuxeo-build

Conversation

@mikejritter

@mikejritter mikejritter commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

What does this do?

  • Move nuxeo config into 2023 folder
  • Add SMTP config to fix startup warnings
  • Remove login config to fix startup warnings
  • Update build properties for new versions of Nuxeo and Tomcat

Why are we doing this? (with JIRA link)
Jira: https://collectionspace.atlassian.net/browse/DRYD-2090

These are updates for our deployment so that it is more consistent with the version of Nuxeo which we've updated to. In addition some config files have been added to handle some warnings which were being logged when Nuxeo starts. It does not address any of the warnings that occur when shutting down Tomcat.

How should this be tested? Do these changes have associated tests?

  • Rebuild and deploy
  • Check either Tomcat's catalina.out or cspace-services.log and see that the previous log entries are no longer found
    • For reference, the smtp config related error was about the JNDI Mail resource: Name [Mail] is not bound in this Context.
    • The login config related error was Warning: target extension point 'domains' of 'org.nuxeo.runtime.LoginComponent' is unknown.

Dependencies for merging? Releasing to production?
No dependencies. I'll run the integration tests again as a sanity check.

Also for releasing to production - I kept these changes pretty minimal. There are some additional components we can configure for Nuxeo which involve: metrics, in memory config for the blob transient storage, and probably some other things. As I mentioned in the related Jira, I think it's better to disable what we don't need by using the configuration rather than trying to omit dependencies.

Has the application documentation been updated for these changes?
I'm going to start working on docs about how we configure Nuxeo and where to find additional information for making changes.

Did someone actually run this code to verify it works?
@mikejritter tested locally.

Have any new security vulnerabilities been handled?
n/a

@mikejritter
mikejritter requested a review from spirosdi June 2, 2026 19:56
@spirosdi

Copy link
Copy Markdown
Contributor

I still see the The login config related error in the logs. Maybe because the login-config.xml is not actually removed but moved to 2023/config?

Comment thread build.properties Outdated
@mikejritter

Copy link
Copy Markdown
Contributor Author

@spirosdi Yep, the login-config was supposed to be deleted. I just removed it now along with the unused build property.

@spirosdi
spirosdi self-requested a review June 17, 2026 14:05

@spirosdi spirosdi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

both warnings are gone now after redeploying.

@mikejritter
mikejritter merged commit eb38436 into collectionspace:develop Jun 18, 2026
@mikejritter
mikejritter deleted the task/dryd-2090-nuxeo-build branch June 18, 2026 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants