-
Notifications
You must be signed in to change notification settings - Fork 12
Update staging compose for demo/preview deployments #1181
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 2 commits
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
8b12cbf
feat: update staging compose for demo/preview deployments
mihow b3d9771
fix: upgrade gunicorn 20.1.0 → 23.0.0
mihow 1bb2646
fix: address PR review feedback
mihow d5e8f6f
fix: remove host port bindings from internal services
mihow 7f75f98
docs: add staging deployment guide
mihow 6b699b7
fix: increase DATA_UPLOAD_MAX_MEMORY_SIZE for ML worker results
mihow 91a7f49
fix(celery): use Redis DB 1 for result backend, separate from cache
mihow c845143
docs(celery): add comments explaining CELERY_RESULT_EXTENDED impact
mihow 14c26f5
fix(redis): add redis.conf, disable bgsave, add CELERY_RESULT_EXPIRES
mihow bd649a9
Merge branch 'main' into feat/update-staging-compose
mihow f12610f
chore(staging): add .compose-example and deploy.sh script
mihow fea3af2
fix(settings): use urllib.parse for Redis DB URL rewriting
mihow 87b8c12
docs(staging): add reverse proxy section with nginx example
mihow 1f2db69
fix(staging): deploy.sh should pull, not just fetch
mihow 47e9fdd
fix(staging): add branch/host echo to deploy.sh before deploying
mihow 095c6fa
docs(staging): clarify staging vs production, explain .envs/.producti…
mihow 9a3b020
docs(staging): note potential rename to demo in future release
mihow 076258f
fix: sort stdlib imports in base.py (isort)
mihow d0a8f1d
fix: unused import
mihow File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,37 @@ | ||
| # Optional local PostgreSQL for staging environments. | ||
| # | ||
| # Use this when you don't have an external database (e.g., for local testing | ||
| # or isolated branch previews). Creates a containerized PostgreSQL instance | ||
| # that the staging compose stack connects to via the Docker network. | ||
| # | ||
| # Usage: | ||
| # # Start the database first | ||
| # docker compose -f compose/staging/docker-compose.db.yml up -d | ||
| # | ||
| # # Then start the app (DATABASE_IP points to the host's Docker bridge) | ||
| # docker compose -f docker-compose.staging.yml --env-file .envs/.production/.compose up -d | ||
| # | ||
| # The app's .envs/.production/.postgres should use POSTGRES_HOST=db and the | ||
| # DATABASE_IP in .envs/.production/.compose should be set to the host IP | ||
| # of the machine running this database container (e.g., 172.17.0.1 for the | ||
| # default Docker bridge, or the host's LAN IP). | ||
| # | ||
| # For ood-style setups where the DB runs on the same host, set: | ||
| # DATABASE_IP=172.17.0.1 (or use host.docker.internal on Docker Desktop) | ||
mihow marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
|
|
||
| volumes: | ||
| staging_postgres_data: {} | ||
|
|
||
| services: | ||
| postgres: | ||
| build: | ||
| context: ../../ | ||
| dockerfile: ./compose/local/postgres/Dockerfile | ||
| volumes: | ||
| - staging_postgres_data:/var/lib/postgresql/data | ||
| - ../../data/db/snapshots:/backups | ||
| env_file: | ||
| - ../../.envs/.production/.postgres | ||
| ports: | ||
| - "5432:5432" | ||
coderabbitai[bot] marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
mihow marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| restart: always | ||
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
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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.