ldraney/pal-e-platform
1
0
Fork 0
Code Issues 104 Pull requests 7 Projects Releases Packages Wiki Activity Actions

fix: add pal-e-docs to postgres NetworkPolicy ingress #286

Open
forgejo_admin wants to merge 1 commit from 284-fix-add-pal-e-docs-namespace-to-postgres into main
pull from: 284-fix-add-pal-e-docs-namespace-to-postgres
merge into: ldraney:main
Conversation 0 Commits 1 Files changed 1 +1
forgejo_admin commented 2026-04-12 18:00:14 +00:00
Contributor
Copy link

Summary

Add pal-e-docs namespace to the postgres default-deny-ingress NetworkPolicy so pal-e-docs pods can reach the CNPG postgres cluster. This was manually patched and needs to be permanent in Terraform.

Changes

  • terraform/network-policies.tf: Added pal-e-docs namespace selector to postgres NetworkPolicy ingress rules

tofu plan Output

Single resource change: kubernetes_manifest.netpol_postgres will be updated in-place to add the pal-e-docs namespace ingress rule.

Test Plan

  • tofu fmt passes (verified)
  • tofu plan -lock=false should show only the postgres NetworkPolicy update
  • After apply, verify pal-e-docs pods can connect to postgres

Review Checklist

  • tofu fmt clean
  • Single-line addition, no other changes
  • Matches manual patch already applied to cluster

Related Notes

None.

Related

Closes #284

## Summary Add `pal-e-docs` namespace to the postgres `default-deny-ingress` NetworkPolicy so pal-e-docs pods can reach the CNPG postgres cluster. This was manually patched and needs to be permanent in Terraform. ## Changes - `terraform/network-policies.tf`: Added `pal-e-docs` namespace selector to postgres NetworkPolicy ingress rules ## tofu plan Output Single resource change: `kubernetes_manifest.netpol_postgres` will be updated in-place to add the `pal-e-docs` namespace ingress rule. ## Test Plan - `tofu fmt` passes (verified) - `tofu plan -lock=false` should show only the postgres NetworkPolicy update - After apply, verify pal-e-docs pods can connect to postgres ## Review Checklist - [x] `tofu fmt` clean - [x] Single-line addition, no other changes - [x] Matches manual patch already applied to cluster ## Related Notes None. ## Related Closes #284
fix: add pal-e-docs to postgres NetworkPolicy ingress
All checks were successful
ci/woodpecker/push/woodpecker Pipeline was successful
Details
ci/woodpecker/pr/woodpecker Pipeline was successful
Details
bb06e9b167 
Closes #284

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
All checks were successful
ci/woodpecker/push/woodpecker Pipeline was successful
Details
ci/woodpecker/pr/woodpecker Pipeline was successful
Details
This pull request has changes conflicting with the target branch.
  • terraform/network-policies.tf
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin 284-fix-add-pal-e-docs-namespace-to-postgres:284-fix-add-pal-e-docs-namespace-to-postgres
git switch 284-fix-add-pal-e-docs-namespace-to-postgres
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
domain:backend

   
domain:devops

   
domain:frontend

   
status:approved

QA passed, awaiting merge approval

  
status:in-progress

Dev agent is actively working

  
status:needs-fix

QA found issues, back to dev

  
status:qa

PR submitted, awaiting QA review

  
type:bug

Bug fix

  
type:devops

Infrastructure/CI/config work

  
type:feature

New functionality

No labels
domain:backend
domain:devops
domain:frontend
status:approved
status:in-progress
status:needs-fix
status:qa
type:bug
type:devops
type:feature
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
ldraney/pal-e-platform!286
No description provided.