stacklok / toolhive / 43232996607

73%

Build Type

push

github

Committed by web-flow

Commit Message

Take a free actor validator slot before waiting (#7125)

actorLimiter.acquire selected between a free slot and the end of the
budget. When both were ready, which happens under load when the budget
is nearly spent, Go picks one at random, so a call could fail as if it
had waited for a slot while slots were free. With a separate log cause
for a call that never got a slot, that sends operators looking for an
overloaded validator that does not exist, and it makes
TestValidateActorTimeoutHonoringContext flaky.

Try the slot without blocking first and wait only when the limiter is
full.

Co-authored-by: Mecatl <noreply@mecatl.dev>

Coverage Stats

3 of 3 new or added lines in 1 file covered. (100.0%)

2 existing lines in 2 files now uncovered.

95435 of 130443 relevant lines covered (73.16%)

889.92 hits per line