feat: consolidate deployment compose into root under --profile deploy (COW-976)#68
Conversation
| timeout: 5s | ||
| retries: 5 | ||
|
|
||
| postgres-deploy: |
There was a problem hiding this comment.
why we can't use the same db container config for prod and dev?
There was a problem hiding this comment.
Good point — addressed in the latest commit. The two services are now unified into a single postgres service that uses ${POSTGRES_USER:-postgres}-style defaults, so docker compose up works for dev without a .env file, and production overrides the credentials via env vars.
️✅ There are no secrets present in this pull request anymore.If these secrets were true positive and are still valid, we highly recommend you to revoke them. 🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request. |
… (COW-976) Moves deployment/docker-compose.yml to the root docker-compose.yml, unifying dev and prod postgres into a single service. Production containers run under the "deploy" profile. Updates manage.ts and deployment docs accordingly. Default postgres credentials are env-var-substituted dev fallbacks, not real secrets. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
8131e6d to
b7f3b58
Compare
Summary
Consolidate
deployment/docker-compose.ymlinto the rootdocker-compose.ymlas a--profile deployprofile, eliminating the duplicate file. The defaultdocker compose updev-database behavior is unchanged.Changes
postgres-deployandponderservices to rootdocker-compose.ymlunderprofiles: [deploy]start-db.shfrom./static/to./deployment/static/(context is now repo root)deployment/docker-compose.ymldeployment/manage.shto reference../docker-compose.ymlwith--profile deploydocs/deployment.mdHow to Test
docker compose --profile deploy upand verify the deploy stack starts (postgres + ponder)docker compose up(no profile) and verify only the dev postgres starts as beforeChecklist
Breaking Changes
None —
docker compose --profile deploy upreplaces the olddocker compose -f deployment/docker-compose.yml up.Related Issues
COW-976