brooktrails/forgejo-runner-operator
Watch
1
0
Fork
You've already forked forgejo-runner-operator
 0
Code Issues 2 Pull requests Projects Releases Packages Wiki Activity Actions

fix(kubernetes): also set DinD MTU on user-defined networks #51

Merged
rcsheets merged 1 commit from fix/dind-mtu-match-pod-network into main 2026-07-20 01:07:14 +00:00
Conversation 2 Commits 1 Files changed 2 +13 -4
rcsheets commented 2026-07-20 01:05:48 +00:00
Owner
Copy link

--mtu only applies to the default bridge (docker0). act_runner runs each
job on a per-job user-defined network, and those default to 1500
regardless of --mtu -- so jobs still emitted oversized frames and large
downloads still stalled even after docker0 was corrected to the pod MTU.

Also pass --default-network-opt bridge=com.docker.network.driver.mtu=$mtu
so every bridge network the daemon creates inherits the matched MTU.
Verified in a runner's dind: a fresh user network's container now comes up
at 1450 instead of 1500.

Co-Authored-By: Claude Opus 4.8 noreply@anthropic.com

--mtu only applies to the default bridge (docker0). act_runner runs each job on a per-job user-defined network, and those default to 1500 regardless of --mtu -- so jobs still emitted oversized frames and large downloads still stalled even after docker0 was corrected to the pod MTU. Also pass --default-network-opt bridge=com.docker.network.driver.mtu=$mtu so every bridge network the daemon creates inherits the matched MTU. Verified in a runner's dind: a fresh user network's container now comes up at 1450 instead of 1500. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
fix(kubernetes): also set DinD MTU on user-defined networks
Some checks failed
CI / ci (pull_request) Has been cancelled
Details
E2E smoke test / e2e (pull_request) Has been cancelled
Details
CI (next Go) / next-go (tip) (pull_request) Has been cancelled
Details
dd9940ef0f 
--mtu only applies to the default bridge (docker0). act_runner runs each
job on a per-job user-defined network, and those default to 1500
regardless of --mtu -- so jobs still emitted oversized frames and large
downloads still stalled even after docker0 was corrected to the pod MTU.

Also pass --default-network-opt bridge=com.docker.network.driver.mtu=$mtu
so every bridge network the daemon creates inherits the matched MTU.
Verified in a runner's dind: a fresh user network's container now comes up
at 1450 instead of 1500.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
pr-reviewer-bot commented 2026-07-20 01:05:50 +00:00
Collaborator
Copy link

Automated review by pr-reviewer v0.32.2 | Safety Check | Mistral Small | tracking id r-5d746e-d63aa7
This is an AI-genrated review and may contain mistakes.

Status: ⚠️ Unavailable

Review unavailable — the inference service isn't reachable right now. This isn't a problem with your PR. Tracking id r-5d746e-d63aa7.

Comment @pr-reviewer-bot retry once it's back.

*Automated review by [pr-reviewer](https://git.brooktrails.org/brooktrails/pr-reviewer) v0.32.2 | Safety Check | Mistral Small | tracking id `r-5d746e-d63aa7`* *This is an AI-genrated review and may contain mistakes.* **Status:** ⚠️ Unavailable --- Review unavailable — the inference service isn't reachable right now. This isn't a problem with your PR. Tracking id `r-5d746e-d63aa7`. Comment `@pr-reviewer-bot retry` once it's back.
rcsheets force-pushed fix/dind-mtu-match-pod-network from dd9940ef0f
Some checks failed
CI / ci (pull_request) Has been cancelled
Details
E2E smoke test / e2e (pull_request) Has been cancelled
Details
CI (next Go) / next-go (tip) (pull_request) Has been cancelled
Details
to e62fb794fd
Some checks failed
CI (next Go) / next-go (tip) (pull_request) Has been cancelled
Details
CI / ci (pull_request) Has been cancelled
Details
E2E smoke test / e2e (pull_request) Failing after 2s
Details
2026-07-20 01:05:58 +00:00 Compare
pr-reviewer-bot commented 2026-07-20 01:05:58 +00:00
Collaborator
Copy link

Automated review by pr-reviewer v0.32.2 | Safety Check | Mistral Small | tracking id r-5d7476-fcf73c
This is an AI-generated review and may contain mistakes.

Status: ⚠️ Unavailable

Review unavailable — the inference service isn't reachable right now. This isn't a problem with your PR. Tracking id r-5d7476-fcf73c.

Comment @pr-reviewer-bot retry once it's back.

*Automated review by [pr-reviewer](https://git.brooktrails.org/brooktrails/pr-reviewer) v0.32.2 | Safety Check | Mistral Small | tracking id `r-5d7476-fcf73c`* *This is an AI-generated review and may contain mistakes.* **Status:** ⚠️ Unavailable --- Review unavailable — the inference service isn't reachable right now. This isn't a problem with your PR. Tracking id `r-5d7476-fcf73c`. Comment `@pr-reviewer-bot retry` once it's back.
rcsheets deleted branch fix/dind-mtu-match-pod-network 2026-07-20 01:07:14 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
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
brooktrails/forgejo-runner-operator!51
No description provided.