coop-cloud/traefik
6
2
Fork 19
Code Issues 14 Pull Requests 6 Releases Wiki Activity

Open ports 5432 for PGSQL new recipe #133

Merged
p4u1 merged 2 commits from pgsql-port-open into master 2026-08-13 14:10:10 +00:00
Conversation 4 Commits 2 Files Changed 2
+16
javielico commented 2026-08-12 15:20:08 +00:00
Owner
Copy Link
Copy Source

I've created a new recipe for a standalone pgsql database so I've included the compose.pgsql.yml file to allow for port 5432 to open on Traefik. Can someone review my PR and let me know if I'm missing anything crucial?

I've created a new recipe for a standalone pgsql database so I've included the compose.pgsql.yml file to allow for port 5432 to open on Traefik. Can someone review my PR and let me know if I'm missing anything crucial? * [X] I have deployed and tested my changes * [ ] I have [updated relevant versions in `abra.sh`](https://docs.coopcloud.tech/maintainers/upgrade/#updating-versions-in-the-abrash) * [ ] I have made my environment variable changes [backwards compatible](https://docs.coopcloud.tech/maintainers/upgrade/#backwards-compatible-environment-variable-changes) * [ ] I have added a [release note entry](https://docs.coopcloud.tech/maintainers/upgrade/#creating-new-release-notes)
javielico added 2 commits 2026-08-12 15:20:10 +00:00
Target port 5432 to open ports for PGSQL Recipe baf02088aa 
This change is for opening port 5432 for recipe pgsql
Add compose.pgsql.yml to compose file
continuous-integration/drone/pr Build is failing
Details
a3416aa31a
mirsal commented 2026-08-12 15:25:07 +00:00
Member
Copy Link
Copy Source

Patch looks OK to me, however, what's the rationale for using traefik instead of exposing that port directly in the postgres recipe?

Patch looks OK to me, however, what's the rationale for using traefik instead of exposing that port directly in the postgres recipe?
❤️ 1
javielico commented 2026-08-12 15:43:37 +00:00
Author
Owner
Copy Link
Copy Source

Patch looks OK to me, however, what's the rationale for using traefik instead of exposing that port directly in the postgres recipe?

Thank you, I've followed the same approach we use for postfix, thinking that I might want to use this recipe to communicate with other apps in the future so I've left it as the same approach.

> Patch looks OK to me, however, what's the rationale for using traefik instead of exposing that port directly in the postgres recipe? Thank you, I've followed the same approach we use for postfix, thinking that I might want to use this recipe to communicate with other apps in the future so I've left it as the same approach.
p4u1 approved these changes 2026-08-12 20:23:26 +00:00
javielico commented 2026-08-13 13:40:37 +00:00
Author
Owner
Copy Link
Copy Source

Thank you @p4u1, am I ok to remove WIP and merge?

Thank you @p4u1, am I ok to remove WIP and merge?
p4u1 marked the pull request as ready for review 2026-08-13 14:09:10 +00:00
p4u1 merged commit 7c8a44bd26 into master 2026-08-13 14:10:10 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
p4u1
Labels
Clear labels
bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 wontfix
This won't be fixed
No labels
Milestone
No items
No Milestone
Assignees
Clear assignees
3wordchant aadil (Aadil Ayub) abra-bot (Abra Bot) ammaratef45 amras (Sarma) BornDeleuze Brooke carla cas (Cassowary) coopcloud decentral1se (d1) fauno (fauno) javielico (Javielico) jjsfunhouse kawaiipunk (KawaiiPunk) knoflook moosemower moritz notplants oxaliq (sorrel) p4u1 renovate-bot (Comrade Renovate Bot) simon stevensting trav (Trav Fryer) yksflip
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: coop-cloud/traefik#133