Skip to content

Fix wrong postgres mount path#769

Open
mueckinger wants to merge 2 commits intothunderbird:mainfrom
mueckinger:main
Open

Fix wrong postgres mount path#769
mueckinger wants to merge 2 commits intothunderbird:mainfrom
mueckinger:main

Conversation

@mueckinger
Copy link
Copy Markdown

@mueckinger mueckinger commented Apr 25, 2026

closes #763


Note

Low Risk
Low risk deployment-config change, but it can affect where Postgres stores data and may require attention when upgrading existing persisted volumes.

Overview
Updates Postgres persistence to mount the volume at /var/lib/postgresql (instead of /var/lib/postgresql/data) in both deploy/docker-compose.yml and the Helm postgres.yaml StatefulSet template.

Also simplifies the backend Docker build by using COPY --chmod=755 for backend-entrypoint.sh, removing the separate chmod layer.

Reviewed by Cursor Bugbot for commit 3a3b687. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 2dd2c5d. Configure here.

Comment thread deploy/docker-compose.yml
POSTGRES_PASSWORD: postgres
volumes:
- pg_data:/var/lib/postgresql/data
- pg_data:/var/lib/postgresql
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pulumi config still uses old postgres mount path

Medium Severity

The mount path fix from /var/lib/postgresql/data to /var/lib/postgresql was applied in docker-compose.yml and postgres.yaml, but deploy/pulumi/src/services.ts still has the old containerPath: '/var/lib/postgresql/data' at line 93. The Pulumi-based deployment will continue to exhibit the same wrong-mount-path issue this PR intends to fix.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 2dd2c5d. Configure here.

@heuzef
Copy link
Copy Markdown

heuzef commented Apr 26, 2026

Workaround tested, working well 👍

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.

dependency failed to start: container deploy-postgres-1 exited

2 participants