Skip to content

fix: attach dokploy-network at Traefik creation in install.sh - #174

Open
Siumauricio wants to merge 1 commit into
mainfrom
fix/install-traefik-network-at-create
Open

fix: attach dokploy-network at Traefik creation in install.sh#174
Siumauricio wants to merge 1 commit into
mainfrom
fix/install-traefik-network-at-create

Conversation

@Siumauricio

@Siumauricio Siumauricio commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Fresh installs created dokploy-traefik without --network and attached dokploy-network afterwards with docker network connect. Dynamic attachments like that are not restored after an ungraceful reboot (VM resize, power loss) — the container fails with failed to set up container networking and --restart always does not retry, which is the root cause behind Dokploy/dokploy#4057 (reproduced on Docker 28.x and 29.x; details in Dokploy/dokploy#4057 (comment)).

Networks defined in the container's create config survive the same crash reliably (verified repeatedly, including a full VM reboot), so the script now passes --network dokploy-network to docker run and drops the separate docker network connect. Safe at install time: the network is already materialized on the node because the postgres and dokploy services are attached to it before Traefik is created.

History note: this reverts f94ed01 (May 2025), which introduced the two-step connect for Dokploy/dokploy#1802 / Dokploy/dokploy#1809. Those reports were a different symptom — Traefik running but not routing after reboot (stale ARP/VIP state, upstream moby/moby#52661) — and the two-step didn't fix reboot recovery: Dokploy/dokploy#3236 and Dokploy/dokploy#4057 (Traefik fully dead after reboot) both happened after that change, caused by the dynamic attachment it introduced.

@Siumauricio
Siumauricio force-pushed the fix/install-traefik-network-at-create branch from 3b98105 to cca80e5 Compare August 9, 2026 07:02
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