Skip to content

docs: update user for postgres healthcheck#7801

Open
poiintt wants to merge 1 commit intoprisma:mainfrom
poiintt:patch-1
Open

docs: update user for postgres healthcheck#7801
poiintt wants to merge 1 commit intoprisma:mainfrom
poiintt:patch-1

Conversation

@poiintt
Copy link
Copy Markdown

@poiintt poiintt commented Apr 15, 2026

The user prisma does not exist in postgres for this docker-compose.
The healtcheck command results in FATAL: role "prisma" does not exist-Logs.
This PR changes the user in the healtcheck command to the correct user postgres, which is configured in the env POSTGRES_USER (line 224)

Summary by CodeRabbit

  • Documentation
    • Corrected PostgreSQL Docker Compose healthcheck configuration in the deployment guide to use the correct database user credentials, ensuring readiness probes validate against the properly configured user.

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 15, 2026

@poiintt is attempting to deploy a commit to the Prisma Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 15, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 6268fe9a-3d95-4a6c-888e-282a8e2db93b

📥 Commits

Reviewing files that changed from the base of the PR and between 9bcb676 and 52fadfe.

📒 Files selected for processing (1)
  • apps/docs/content/docs/guides/deployment/docker.mdx

Walkthrough

A single line correction to a Docker Compose configuration example in the deployment guide. The PostgreSQL healthcheck command now uses the correct username (postgres) that matches the configured POSTGRES_USER environment variable, replacing an incorrect reference to a non-existent prisma user.

Changes

Cohort / File(s) Summary
Docker Deployment Guide
apps/docs/content/docs/guides/deployment/docker.mdx
Updated PostgreSQL healthcheck command to use the correct username (postgres) instead of the non-existent prisma user, ensuring the readiness probe aligns with the actual service configuration.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 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 directly and accurately describes the main change: updating the PostgreSQL healthcheck username from 'prisma' to 'postgres' in the Docker documentation.
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.


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.

❤️ Share

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

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