chore: Bump examples to use PG18 - BED-8581#28
Conversation
WalkthroughPostgreSQL is upgraded from version 16 to 18 across both development and production docker-compose configurations. The persistent storage volume mount path is updated from ChangesPostgreSQL Service Configuration
Estimated code review effort🎯 2 (Simple) | ⏱️ ~5 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docker-compose.dev.yml`:
- Line 47: The volume mount uses ${BH_POSTGRES_VOLUME:-postgres-data} at
/var/lib/postgresql which may not match the image PGDATA; update
docker-compose.dev.yml to explicitly set PGDATA and/or change the volume target
to the image's default (e.g., set environment variable
PGDATA=/var/lib/postgresql/18/docker and mount
${BH_POSTGRES_VOLUME:-postgres-data}:/var/lib/postgresql/18/docker or add a
secondary mount) so data is reused across upgrades, and add a short migration
note in docs explaining how to copy existing data from /var/lib/postgresql (old
volume) into the new PGDATA path (/var/lib/postgresql/18/docker) before starting
the container.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 52040475-0dee-4c07-affc-ec2817b26654
📒 Files selected for processing (2)
docker-compose.dev.ymldocker-compose.yml
The BHE hosted fleet runs entirely on PG18. We are moving to deprecate PG16 support and migrating new deployments to PG18 ahead of that.
Summary by CodeRabbit