chore: Upgrade local & dev postgres to 18#264
Conversation
Matches version deployed in external database service after upgrade.
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughThis pull request updates PostgreSQL from version 16.9 to 18 across infrastructure configurations. The Ansible role's default image variable is updated, and the Docker Compose configuration adjusts both the PostgreSQL image reference and modifies the database volume mount path. Changes
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@infra/local/docker-compose.yml`:
- Line 108: Docker Compose mounts Postgres data to /var/lib/postgresql which
mismatches the Ansible role expecting /var/lib/postgresql/data; update the
docker-compose service volume entry (the string
"shared_database_data:/var/lib/postgresql") to
"shared_database_data:/var/lib/postgresql/data" so both environments use the
same data directory and avoid initialization discrepancies.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 05fef444-19b3-4c1b-819e-7de5f4c0dbe6
📒 Files selected for processing (2)
infra/ansible/roles/postgres/defaults/main.ymlinfra/local/docker-compose.yml
Summary
Matches version deployed in external database service after upgrade.
Change in mount path due to error message on first boot with original mount path:
Error: in 18+, these Docker images are configured to store database data in a
format which is compatible with "pg_ctlcluster" (specifically, using
major-version-specific directory names). This better reflects how
PostgreSQL itself works, and how upgrades are to be performed.
Summary by CodeRabbit