janpeter/Scrum4Me
0
0
Fork 0
Code Issues 3 Pull requests 2 Projects Releases Packages Wiki Activity Actions 1

ci: Postgres-poort van de ci-job niet meer op de runner-host publiceren #305

Merged
janpeter merged 1 commit from ci/drop-ci-postgres-host-port into main 2026-10-05 10:33:06 +02:00
Conversation 0 Commits 1 Files changed 1 -2
janpeter commented 2026-10-05 10:11:33 +02:00
Owner
Copy link

De ci-job publiceerde zijn Postgres-service op de runner-host (ports: 5432:5432), terwijl hij de database via de servicenaam bereikt (postgres:5432). De andere jobs met een Postgres-service doen dat al niet.

Waarom: runs 867 en 868 (PR #301 en #302 vlak na elkaar gemerged) draaiden tegelijk; 868 faalde met Bind for 0.0.0.0:5432 failed: port is already allocated, waardoor main rood werd zonder codefout.

Verificatie: npm run verify lokaal groen op 678f8668; de CI van deze PR toont dat de ci-job zonder host-mapping draait.

🤖 Generated with Claude Code

De `ci`-job publiceerde zijn Postgres-service op de runner-host (`ports: 5432:5432`), terwijl hij de database via de servicenaam bereikt (`postgres:5432`). De andere jobs met een Postgres-service doen dat al niet. **Waarom:** runs 867 en 868 (PR #301 en #302 vlak na elkaar gemerged) draaiden tegelijk; 868 faalde met `Bind for 0.0.0.0:5432 failed: port is already allocated`, waardoor `main` rood werd zonder codefout. **Verificatie:** `npm run verify` lokaal groen op `678f8668`; de CI van deze PR toont dat de ci-job zonder host-mapping draait. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
ci: publiceer de Postgres-poort van de ci-job niet meer op de runner-host
All checks were successful
CI / Lint, Typecheck, Test & Build (pull_request) Successful in 9m7s
Details
CI / DB Access Policy Gate (pull_request) Successful in 2m32s
Details
CI / Queue Dispatch Guard Gate (pull_request) Successful in 2m36s
Details
CI / DB Access Production Sentinel (pull_request) Successful in 1m59s
Details
CI / Deploy Manual (workflow_dispatch) (pull_request) Has been skipped
Details
CI / Detect deploy-relevant changes (pull_request) Has been skipped
Details
CI / Deploy Preview (PR) (pull_request) Has been skipped
Details
CI / Deploy Production (main) (pull_request) Has been skipped
Details
4378a909f1 
De ci-job bereikt Postgres via de servicenaam (postgres:5432); de host-
mapping 5432:5432 was overbodig. Twee gelijktijdige runs (867/868, PR #301
en #302 vlak na elkaar gemerged) botsten erop: "Bind for 0.0.0.0:5432
failed: port is already allocated". De andere jobs met een Postgres-
service hebben de mapping al niet.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
janpeter scheduled this pull request to auto merge when all checks succeed 2026-10-05 10:14:07 +02:00
s4m-codex-reviewer approved these changes 2026-10-05 10:16:51 +02:00
s4m-codex-reviewer left a comment
Copy link

APPROVED

Findings:

  • INFO — .github/workflows/ci.yml:33 — Het verwijderen van 5432:5432 voorkomt de gedeelde hostpoortclaim. De teststap gebruikt postgres:5432 (regel 74); servicecommunicatie vereist bij Forgejo geen hostpoortmapping (Forgejo Services). Dit sluit aan bij de overige databasejobs in dezelfde workflow. Healthcheck en testconfiguratie blijven intact. Geen blokkerende bevindingen.
  • INFO — .github/workflows/ci.yml:70 — CI-run 873 voor 4378a909f159a88920b8d22a4d8697e2890829a8 staat nog op pending. Tests/build en gelijktijdige uitvoering zijn dus nog niet bevestigd; geen tests lokaal uitgevoerd. De bestaande database-teststap biedt relevante verificatie zodra de CI draait.

De wijziging is minimaal en past bij de productrichtlijn om de oorzaak te verhelpen met de kleinste passende wijziging. De PR-beschrijving documenteert oorzaak en bedoeling voldoende; voor deze configuratiecorrectie is geen nieuwe architectuurdocumentatie nodig.

geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

# APPROVED Findings: - **INFO — .github/workflows/ci.yml:33** — Het verwijderen van `5432:5432` voorkomt de gedeelde hostpoortclaim. De teststap gebruikt `postgres:5432` (regel 74); servicecommunicatie vereist bij Forgejo geen hostpoortmapping ([Forgejo Services](https://forgejo.org/docs/v17.0/user/actions/advanced-features/#services)). Dit sluit aan bij de overige databasejobs in dezelfde workflow. Healthcheck en testconfiguratie blijven intact. Geen blokkerende bevindingen. - **INFO — .github/workflows/ci.yml:70** — CI-run 873 voor `4378a909f159a88920b8d22a4d8697e2890829a8` staat nog op pending. Tests/build en gelijktijdige uitvoering zijn dus nog niet bevestigd; geen tests lokaal uitgevoerd. De bestaande database-teststap biedt relevante verificatie zodra de CI draait. De wijziging is minimaal en past bij de productrichtlijn om de oorzaak te verhelpen met de kleinste passende wijziging. De PR-beschrijving documenteert oorzaak en bedoeling voldoende; voor deze configuratiecorrectie is geen nieuwe architectuurdocumentatie nodig. geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.
Sign in to join this conversation.
Reviewers
No reviewers
s4m-codex-reviewer
Labels
Clear labels   
severity/s3

   
severity/s4
No labels
severity/s3
severity/s4
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
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
janpeter/Scrum4Me!305
No description provided.