Skip to content

Fix ExecStartPre ordering: run routing before prestart#200

Merged
mairas merged 2 commits into
mainfrom
fix/prestart-ordering
Apr 8, 2026
Merged

Fix ExecStartPre ordering: run routing before prestart#200
mairas merged 2 commits into
mainfrom
fix/prestart-ordering

Conversation

@mairas
Copy link
Copy Markdown
Contributor

@mairas mairas commented Apr 8, 2026

Summary

  • Move configure-container-routing before prestart.sh in the systemd service template's ExecStartPre chain
  • Ensures the port registry is populated before prestart scripts read it

Without this fix, any prestart script that reads the port registry (currently Grafana and Signal K) either gets the wrong port on first boot or enters a permanent restart loop if it validates the port.

Test plan

  • All 903 existing tests pass
  • Verify on fresh image: container apps start on first boot without restart loop
  • Verify OIDC redirect URIs use the correct assigned port on first boot

🤖 Generated with Claude Code

mairas and others added 2 commits April 8, 2026 14:48
Move configure-container-routing ahead of prestart.sh in the
ExecStartPre chain so that the port registry is populated before
prestart scripts read it. Previously, prestart ran first and could
not find its assigned port on first boot, causing either silent
misconfiguration (wrong port) or a permanent restart loop if
prestart validates the port.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@mairas mairas merged commit 422e798 into main Apr 8, 2026
4 checks passed
@mairas mairas deleted the fix/prestart-ordering branch April 8, 2026 11:58
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