Skip to content

fix(compose): fix volume mount for postgres for PG18#308

Merged
jsell-rh merged 113 commits intomainfrom
chore/update-container-images
Apr 10, 2026
Merged

fix(compose): fix volume mount for postgres for PG18#308
jsell-rh merged 113 commits intomainfrom
chore/update-container-images

Conversation

@jsell-rh
Copy link
Copy Markdown
Collaborator

@jsell-rh jsell-rh commented Apr 10, 2026

Summary by CodeRabbit

Chores

  • Updated Docker Compose configuration for PostgreSQL database services. The data storage volume mount has been repositioned to optimize how database data is persisted across container restarts and redeployments. This infrastructure adjustment ensures more reliable data preservation in deployment environments.

…lembic

- Add SQLAlchemy 2.0 with asyncpg for async database operations
- Add Alembic for schema migrations
- Add python-ulid for ULID support instead of UUID
- Create read/write engine separation with connection pooling
- Create FastAPI dependency injection for database sessions
- Create SQLAlchemy declarative base with timestamp mixin
- Initialize Alembic with async migration support
- Create initial migration for teams table (ULID primary key)
- Add comprehensive unit tests for engines and dependencies
- Configure Alembic to use settings module for database URL
- Enable ruff post-write hook for migration formatting

Refs: AIHCM-121
- Add authzed library for SpiceDB integration
- Add python-ulid for ULID support
- Create ResourceType, RelationType, Permission enums (using Group not Team)
- Create AuthorizationProvider protocol for swappable implementations
- Implement SpiceDBClient with async methods for relationships and permissions
- Create SpiceDB schema (.zed) with Tenant→Workspace→Group hierarchy
- Create AuthorizationProbe for domain-oriented observability
- Move ObservationContext to shared_kernel (fix architectural boundary)
- Add 35 unit tests for types and probes
- All 410 tests passing

Refs: AIHCM-122
Resolved conflicts in authorization files by accepting remote changes:
- shared_kernel/authorization/types.py (docstring fix)
- shared_kernel/authorization/spicedb/client.py (_parse_reference helper)
jsell-rh and others added 25 commits February 18, 2026 11:40
- apache/age: release_PG17_1.6.0 → release_PG18_1.7.0
- authzed/spicedb: v1.48.0 → v1.50.0
- ubi9/python-312: pin to digest sha256:d7b4607a...

Co-Authored-By: Claude Opus 4.6 <[email protected]>
…o PG18

- Pin spicedb-migrate to v1.50.0 to match spicedb service
- Update db-init job and compose postgres client from 17-alpine to 18-alpine

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 10, 2026

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

Updated the Docker Compose compose.yaml file to modify the PostgreSQL service's volume mount configuration. The named volume postgres_data is now mounted to /var/lib/postgresql instead of /var/lib/postgresql/data, changing the internal container path where database files are persisted.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies a specific fix to the Docker Compose postgres service volume mount configuration related to PostgreSQL 18, matching the actual changeset which modifies the volume mount path.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/update-container-images

Comment @coderabbitai help to get the list of available commands and usage tips.

@jsell-rh jsell-rh merged commit 1f3008d into main Apr 10, 2026
9 of 10 checks passed
@jsell-rh jsell-rh deleted the chore/update-container-images branch April 10, 2026 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant