Skip to content

Fix Finance database network resolution - #9

Merged
iPior merged 1 commit into
mainfrom
fix/remove-finlens-private-network
Aug 7, 2026
Merged

Fix Finance database network resolution#9
iPior merged 1 commit into
mainfrom
fix/remove-finlens-private-network

Conversation

@iPior

@iPior iPior commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • remove the obsolete finlens_private network from the API service
  • remove the obsolete finlens_private network from the MCP server
  • remove the external finlens_private network declaration
  • keep pior_data as the shared database network

Why

Finance was resolving the database hostname postgres to the legacy shared-postgres container on finlens_private (172.23.0.2) instead of the platform-managed pior-postgres container on pior_data (172.24.0.2). This caused the migration to fail with PostgreSQL 28P01 even though the finance_app credentials were correct.

A migration run with the database hostname temporarily overridden to pior-postgres succeeded, confirming the network/DNS collision as the root cause.

@iPior
iPior merged commit ee6c12a into main Aug 7, 2026
2 checks passed
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