Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ 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.
| POSTGRES_PASSWORD: postgres | ||
| volumes: | ||
| - pg_data:/var/lib/postgresql/data | ||
| - pg_data:/var/lib/postgresql |
There was a problem hiding this comment.
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)
Reviewed by Cursor Bugbot for commit 2dd2c5d. Configure here.
|
Workaround tested, working well 👍 |


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 bothdeploy/docker-compose.ymland the Helmpostgres.yamlStatefulSet template.Also simplifies the backend Docker build by using
COPY --chmod=755forbackend-entrypoint.sh, removing the separatechmodlayer.Reviewed by Cursor Bugbot for commit 3a3b687. Bugbot is set up for automated code reviews on this repo. Configure here.