Skip to content

Log rejection of polls from recently-shutdown workers#9811

Merged
rkannan82 merged 5 commits intomainfrom
kannan/log-shutdown-poll-rejection
Apr 7, 2026
Merged

Log rejection of polls from recently-shutdown workers#9811
rkannan82 merged 5 commits intomainfrom
kannan/log-shutdown-poll-rejection

Conversation

@rkannan82
Copy link
Copy Markdown
Contributor

@rkannan82 rkannan82 commented Apr 3, 2026

What

Add an Info-level log when a poll is rejected because the worker is in the shutdown cache. Follows up on #9545.

Why

The shutdown-worker cache returns errNoTasks silently, making cache rejections indistinguishable from normal empty polls. This log helps operators debug cases where a worker's polls aren't getting tasks post-shutdown.

How did you test it?

Built.

The shutdown-worker cache silently returns errNoTasks, making it
indistinguishable from a normal empty poll. Add an Info log so
operators can tell when zombie re-polls are being rejected.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@rkannan82 rkannan82 requested a review from a team as a code owner April 3, 2026 18:32
@rkannan82 rkannan82 requested a review from ShahabT April 3, 2026 18:38
@rkannan82 rkannan82 enabled auto-merge (squash) April 3, 2026 18:38
@rkannan82 rkannan82 requested a review from dnr April 7, 2026 00:35
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@rkannan82 rkannan82 requested a review from dnr April 7, 2026 01:22
@rkannan82 rkannan82 merged commit 832a576 into main Apr 7, 2026
46 checks passed
@rkannan82 rkannan82 deleted the kannan/log-shutdown-poll-rejection branch April 7, 2026 02:30
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.

2 participants