fix(compose): drop db host port publish to avoid 5432 clash#99
Merged
Conversation
Host may already run Postgres on 5432; publishing the db container port collided and crashed the stack. App reaches db over the internal compose network (db:5432), so the host publish was unnecessary.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
docker compose upcrashed the stack on hosts already running Postgres on5432. Thedbservice published5432:5432; on recreate the host port bind failed (address already in use), db died, app died withP1001: Can't reach database server at db:5432.Fix
Remove the db host port publish. The app reaches the database over the internal compose network (
db:5432), so publishing to the host was unnecessary. Local Postgres on the host is left untouched.Trade-off
Host tools (psql/pgAdmin from host) can no longer reach the container db directly. Add a non-conflicting publish (e.g.
5433:5432) locally if needed.🤖 Generated with Claude Code