Skip to content

Restore "Long running tasks" logging when the dynamic UI is disabled - #23612

Open
tobni wants to merge 2 commits into
pantsbuild:mainfrom
tobni:fix/straggler-drain
Open

Restore "Long running tasks" logging when the dynamic UI is disabled#23612
tobni wants to merge 2 commits into
pantsbuild:mainfrom
tobni:fix/straggler-drain

Conversation

@tobni

@tobni tobni commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Re-introduces the "Long running tasks ..." log in CI environments, which I accidentally nuked when disabling the workunit message queues that were "never" polled.

This change instead drains said queue regularly (to avoid garbage), and logs if there are indeed stragglers during CI runs, which reportedly some environments depend on for keep-alive, see #23604.

Closes #23603

tobni added 2 commits August 8, 2026 14:56
Both display modes consume that queue, so only enabling it for the dynamic UI stopped "Long running tasks:" from ever being logged. Draining it from the render tick instead of from its consumers keeps it bounded, which is what gating it was for.
@tobni tobni added this to the 2.33.x milestone Aug 8, 2026
@tobni tobni added needs-cherrypick [CI] category:bugfix Bug fixes for released features labels Aug 8, 2026
@tobni
tobni requested a review from cburroughs August 8, 2026 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:bugfix Bug fixes for released features needs-cherrypick [CI]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"Long running tasks" lines no longer printed on v.2.33.0, when dynamic_ui = false

2 participants