Add Render deploy Blueprint#2314
Closed
Ho1yShif wants to merge 6 commits into
Closed
Conversation
Render's envVarGroups don't accept sync: false placeholders, which silently dropped FIRST_SUPERUSER, FIRST_SUPERUSER_PASSWORD, FRONTEND_HOST, BACKEND_CORS_ORIGINS, SMTP credentials, EMAILS_FROM_EMAIL, SENTRY_DSN, and VITE_API_URL on apply. Move each to its consuming service (backend or frontend) and keep the env group for shared, non-secret defaults only. Also move SECRET_KEY (backend-only) to the backend service and drop the unused DOMAIN variable. README setup steps updated to point users at the correct service Environment pages. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Upstream deleted backend/scripts/start.sh and split its responsibilities: the Dockerfile CMD now runs fastapi directly, and prestart.sh runs in a separate compose service. Render has no equivalent of "wait for sidecar to complete," and the existing Render service is configured with `bash scripts/start.sh` as its Docker command. Restore the file with its prior contents so the deploy works without any Dashboard config changes. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Render auto-sets WEB_CONCURRENCY based on the instance's CPU/memory (1 on the free tier). Hardcoding --workers 4 caused workers to be OOM-killed on the free tier — the parent kept respawning replacements, producing a steady stream of "Child process died" messages and wasted resources. Default to 1 if WEB_CONCURRENCY is unset. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…late For upstream PR — if merged, the Blueprint lives in fastapi/..., so the deploy button and render.yaml repo: field should point there. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Author
|
Closing to reopen from a more clearly-named branch (Ho1yShif:fastapi). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
render.yaml— Render Blueprint for one-click deploy (Docker backend, static frontend, managed Postgres, shared env group).backend/scripts/start.sh— restore the single-command startup script. Render's container model has no equivalent of compose's separateprestartservice (no "wait for sidecar to complete"), so combiningprestart.shandfastapi runinto one script is the cleanest fit for Render's Docker command.README.md— adds a "Deploy to Render" section with the button and setup steps.Local Docker Compose workflows are unchanged.
Test plan
prestart.sh, then serves on$PORT.VITE_API_URLbaked in and serves the SPA./api/v1/utils/health-check/returns 200.