4
38%
master: 38%

Ran 27 Mar 2026 01:36AM UTC

Files 73

Run time 2s

Badge

Embed ▾

Committed 27 Mar 2026 01:31AM UTC coverage: 46.185%. Remained the same

Job # 23626685081.4

Build Type

push

github

Committed by

web-flow

Commit Message

Fix goroutine leak in monitor workOne due to unbuffered channel (#4714)

* Fix goroutine leak in monitor workOne due to unbuffered channel

When the 50s context timeout fires, workOne returns but the execute
goroutine blocks forever on sending to the unbuffered allJobsDone
channel, since no receiver is waiting anymore. This leaks the goroutine
and the entire Monitor struct (K8s clientsets, HTTP connection pools,
TLS state) on every timeout.

Buffering the channel with capacity 1 allows execute to send and exit
even when workOne has already returned.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Add regression test for execute goroutine leak on timeout path

TestExecuteReturnsWhenNoReceiver simulates the workOne timeout scenario:
context is cancelled and nobody reads from the done channel. With the
previous unbuffered channel, execute would block forever on done <- true.
With the buffered channel fix, execute returns promptly.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Coverage Stats

2778 of 6015 relevant lines covered (46.18%)

0.5 hits per line

Azure / ARO-RP / 23626685081 / 4
38%
master: 38%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job pkg-operator - 23626685081.4

Azure / ARO-RP / 23626685081 / 4 38% master: 38%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job pkg-operator - 23626685081.4

Azure / ARO-RP / 23626685081 / 4
38%
master: 38%

README BADGES
x