bug(envd): HTTP listener bound after blocking init chain — connect-refused during startup race
Nobody has claimed this yet.
- Dominant language
- Go
- Stars
- 1.6k
- Forks
- 438
- PR merge metrics
- No merged PRs in 30d
Description
Summary
On fresh cold boot, run() in packages/envd/main.go calls s.ListenAndServe() as the last statement after a long synchronous init chain. Any blocking call in that chain leaves the process alive but port :49983 returning ECONNREFUSED — indistinguishable from a crash to any TCP-based health check.
Root Cause
run() startup order (main.go):
1. go PollForMMDSOpts(...) ← goroutine, async
2. logs.NewLogger(...) ← sync
3. filesystemRpc.Handle(...) ← sync
4. createCgroupManager() ← sync + kernel I/O ← most likely hang point
5. NewWorkloadFreezer(...) ← sync
6. processRpc.Handle(...) ← sync
7. api.New(...) ← sync
8. publicport.NewScanner/Forwarder ← sync
9. s.ListenAndServe() ← TCP port bound HERE, too late
Most likely blocking point: createCgroupManager() → createCgroup() → writeCgroupProp() (cgroup2.go:117) writes to cgroupv2 files (memory.high, memory.max) via os.OpenFile + WriteString with no timeout. Under host memory pressure the kernel blocks the write() syscall during page reclaim. The process stays alive (context not cancelled, goroutines running) but :49983 returns ECONNREFUSED.
Verified on dev (main branch, Linux 6.8 cgroupv2 host)
Instrumented reproduction with a 200 ms synthetic init delay:
Current behavior (listener-last):
[10ms] port CLOSED — process alive, ECONNREFUSED ← RACE WINDOW
[100ms] port CLOSED — process alive, ECONNREFUSED ← RACE WINDOW
[190ms] port CLOSED — process alive, ECONNREFUSED ← RACE WINDOW
[201ms] port OPEN — ListenAndServe finally reached
Fixed behavior (listener-first):
[0ms] port OPEN — kernel queues SYNs immediately
[10ms] port OPEN — still open during entire init
[201ms] port OPEN — Serve() starts accepting
Relation to #3609 and #3559
This is the mechanism behind #3609 (listener-last race). The two issues are independent:
- This bug fires before the HTTP listener binds (PostInit is never called)
- #3559 fires after the listener is up (PostInit is called repeatedly)
The "5-15 s MMDS traffic" described in #3609 does not match the current PollForMMDSOpts implementation (one-shot, exits after first success), suggesting the reporter observed an older version or an external MMDS source.
Fix
Call net.Listen() as the first operation in run(), before any blocking initializer. Replace s.ListenAndServe() with s.Serve(ln). Add stderr phase logs around createCgroupManager so a hung init is visible in logs.
PR with the fix: coming separately.
Impact
- Rare (< 1 % of cold boots per #3609) but permanent — the stuck sandbox never recovers since envd never exits
- Affects any orchestrator that TCP-dials
:49983as a readiness signal - Self-hosted deployments are most exposed (no automatic node recycling)
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Read packages/envd/main.go and trace run() through createCgroupManager; inspect cgroupv2.go:117 to understand the blocking initialization path. Reproduce the 200 ms init delay and verify the listener binds before initialization, while phase logs make a stalled cgroup setup visible.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go, linux
- Domain
- infrastructure, networking
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Active
- Clarity
- Clearly specified
- Newbie friendliness
- 68/100