Skip to content

Conversation

@jonbartels
Copy link
Contributor

…h and other forks

@jonbartels jonbartels force-pushed the update-readme-with-compatibility-info branch from 33d785d to 7dd968f Compare July 3, 2025 13:48
@jonbartels jonbartels requested review from a team, gibson9583, kayyagari, kpalang, pacmano1, ssrowe and tonygermano and removed request for a team July 3, 2025 13:52
@jonbartels jonbartels force-pushed the update-readme-with-compatibility-info branch from 7dd968f to 5248dc2 Compare July 3, 2025 13:54
kayyagari
kayyagari previously approved these changes Jul 3, 2025
Copy link

@pacmano1 pacmano1 left a comment

Choose a reason for hiding this comment

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

You need to talk about vmoptions.

Copy link
Member

@tonygermano tonygermano left a comment

Choose a reason for hiding this comment

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

This section reads more like a general server migration guide than docker fork compatibility. It's asking people to do a lot to switch to our docker container if they are already using the NG one.

I expect most people to be setting mirth.properties values via environment variables. As best as I can tell that hasn't changed and should be compatible. If they are pointing this container to an existing database, then none of the export/imports should be relevant. As always, the software must be pointed to a database at the same version level or lower in order for database migration to work, which automatically excludes Mirth 4.6+ and BridgeLink at this time (as they started at version 4.5.3 and we are still at 4.5.2.)

Relevant to current users of the NextGen containers version 4.5.2 and earlier is how they can use existing docker volumes with the new container, or if that is not supported at this time (I think it should be possible to do and would make upgrades much smoother.)

@jonbartels jonbartels force-pushed the update-readme-with-compatibility-info branch from 5248dc2 to 85d2b89 Compare July 3, 2025 15:21
1. Update your Mirth Connect installation to Mirth Connect 4.5.2
2. Backup your Mirth Connect database
3. Stop your Mirth Connect server
4. Update Open Integration Engine to use the same `vmoptions` and `mirth.properties` files as your Mirth Connect installation. Containerized instances will use environment variables to set these options. Other installations use the config files.
Copy link

@leovander leovander Jul 3, 2025

Choose a reason for hiding this comment

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

Containerized instances will use environment variables to set these options.

Is that generally true? It's the tiniest nit in the world, but the docker hub connect hub page/readme has the following:

You can use environment variables to configure the mirth.properties⁠ file or to add custom JVM options.

e.g. shall/should/must.

That is the ideal way, but couldn't a containerized version also read from a mirth.properties file?

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.

6 participants