update docs to suggest using Docker in production#11862
update docs to suggest using Docker in production#11862stevenwinship merged 1 commit intodevelopfrom
Conversation
This comment has been minimized.
This comment has been minimized.
|
Changes unknown |
|
📦 Pushed preview images as 🚢 See on GHCR. Use by referencing with full name as printed above, mind the registry name. |
|
During today's Containerization working group meeting I talked through this PR with @poikilotherm @beepsoft and @akio-sone. The recording is here: https://harvard.zoom.us/rec/share/qw5BTHsX132xEYBue7rKrPx6n7_3oL1J5AoOz40y9STlp_z_cuHM2qbTcOtWuaDG.34EEw9prY2HxDp9c In short, I'm removing the scary language about never using these containers in production. @johannes-darms is using them, and others. @poikilotherm rightly pointed out that not everything can be configured via MPConfig, such as storage, but we have an issue tracking this: There will always be something to improve. We have tagged versions now, at least, so people can pin their installations to a specific version. I think it's time. I'm moving this to "ready for review". I'm happy to make changes, of course! |
What this PR does / why we need it:
Now that we have images tagged with version numbers, I think we're ready to suggest that people can try using Docker in production
Here's a good entry point for previewing the changes: https://dataverse-guide--11862.org.readthedocs.build/en/11862/installation/prep.html#choose-your-own-installation-adventure
Which issue(s) this PR closes:
None.
Special notes for your reviewer:
I removed old notes about IQSS not supporting these Docker images at all. There were added years ago by @poikilotherm when he was inventing all this stuff. These days, we use the Docker images in development and we at least talk about using them in production. I think we can give our usual "best effort" attempt at answering questions about them.
See also discussion on Zulip about this PR starting here: https://dataverse.zulipchat.com/#narrow/channel/375812-containers/topic/containers.20for.20production/near/542761006
This issue is related but I think we can work on it in a future PR:
Is there a release notes update needed for this change?:
I don't think so. I'm ok with a soft launch. No need to shout from the mountain tops that we removed some of the "not for production" language.