Update Docker documentation with production setup and troubleshooting#3109
Update Docker documentation with production setup and troubleshooting#3109pwizla wants to merge 16 commits into
Conversation
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Review changes: remove unverified comments, rewrite ambiguous danger admonition and move to troubleshooting, reorder DB tabs (PostgreSQL first), fix contradictory env vars note, document statically compiled admin panel vars, comment out yarn build in dev Dockerfile, comment out platform linux/amd64 on DB services, add upload persistence warning, add secrets management tip, add JS/TS tabs for database config, improve intro with Docker rationale and community tools pointer, align community H3 headings. Integrity fixes: add missing ENCRYPTION_KEY to .env and snippet table, add API_TOKEN_SALT and TRANSFER_TOKEN_SALT to snippet, remove invalid URL env var from FAQ, fix deprecated npm --only=production flag, update GCP Container Registry to Artifact Registry, remove unnecessary nasm package from Alpine dependencies. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Move Community tools before Troubleshooting for better reading flow. Move Docker FAQ entries to the global /cms/faq page. Replace Next steps with the Icon-based "What to do next?" format used in the quick start. Remove nasm from Alpine dependencies (unnecessary for Sharp 0.33.x). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Align with PR #3042 (Remove yarn from main pages). Keep npm only for Dockerfiles, docker-compose volumes, prerequisites, and prose. Also remove em dash from env vars note. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Use the Checklist/ChecklistItem custom component for the production tips (database ports, upload persistence, secrets management) instead of 3 consecutive admonitions. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Add STRAPI_ADMIN_BACKEND_URL ARG to production Dockerfile - Move commented RUN build to a proper tip callout - Remove fragile <br/> in tip callout - Update installation.md intro to match new Docker page scope - Split dense FAQ paragraph into readable chunks - Add tip about optimizing production image size with selective COPY - Clarify npm prerequisite with yarn/pnpm mention - Improve ENCRYPTION_KEY description in env table - Add comments on exposed DB ports in dev docker-compose - Add code example for pool timeout settings in troubleshooting - Add note about nasm removal from Alpine packages
|
I see that docusaurus/static/llms-code.txt has some discrepancies like the yarn example for dockerfiles. Do you want to keep that so AI's have a yarn example even though we don't show it in the docs? |
Thanks for the comment, @OcneanuVlad! Actually they shouldn't be included in PRs, I have another open PR that will fix this long-standing issue. It's just that for now, they are also created locally, so you can check them on the local server, but for some reason, if I completely exclude them (.gitignore) the Vercel build breaks 😅 . |
Warn readers not to set NODE_ENV=production before npm ci, as npm would skip devDependencies needed for the admin build. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Show how to use apk --virtual .build-deps to reduce image size in the single-stage development Dockerfile. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add a concrete docker exec test command as the first step, distinguishing missing library vs. glibc/musl mismatch errors. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Update the selective copy examples to mention the correct path where the Strapi 5 admin bundle lands. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add inline comment to help users migrating Dockerfiles from older Strapi guides that used the now-deprecated npm flag. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Thanks for the review, @derrickmehaffy! Very helpful. Everything should be addressed now. |
This PR rewrites the Docker documentation page to address user feedback about outdated content.
🤖 Generated with Claude Code
Direct preview link 👉 here