Skip to content

revert(pr-poller): stop auto-dispatching bluefin, lts, and dakota - #631

Open
castrojo wants to merge 1 commit into
mainfrom
lab/revert-auto-repos
Open

revert(pr-poller): stop auto-dispatching bluefin, lts, and dakota#631
castrojo wants to merge 1 commit into
mainfrom
lab/revert-auto-repos

Conversation

@castrojo

@castrojo castrojo commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

Reverts the AUTO_REPOS half of #630.

The container QA lane these dispatch into fails 100% of the time — 44 consecutive runs, zero passes. Auto-dispatching three more repositories into it only paints a failing ghost-lab status on every open PR, which inflates the review queue it was meant to drain.

#619 (uinput + setuptools<81) and #618 (dconf TOCTOU) fix the two known causes and are live on the cluster, but no lane has been observed passing end to end yet. Restore these repos once one has.

test-on-lab still opts an individual PR in.

Verified: just lint passes, embedded script passes bash -n.

Adding these in #630 was premature. The container QA lane they dispatch
into fails 100% of the time -- 44 consecutive runs, zero passes -- so the
only effect was a failing ghost-lab status on every open pull request in
three more repositories, on top of the ones common and testsuite were
already producing.

That is the opposite of the intent: it inflates the review queue it was
meant to drain, and a red status nobody can act on trains maintainers to
ignore the signal.

#619 and #618 fix the two known causes (missing pkg_resources under
Python 3.14, and /dev/uinput permissions blocking dogtail input
synthesis). Restore these repos once a lane is observed passing end to
end, not before. test-on-lab still opts an individual pull request in.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

@hanthor hanthor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Diff is a clean, symmetric revert of the AUTO_REPOS addition from #630 (just the three entries removed, comment updated to explain why) — no collateral changes. CI (lint) is green.

The justification is concrete and falsifiable: 44/44 consecutive failures on the container QA lane these three repos would dispatch into, which was turning every open PR's ghost-lab status red and inflating the review queue rather than draining it. The two suspected root causes (#619 uinput/setuptools, #618 dconf TOCTOU) are named as merged but not yet observed passing end-to-end in the lane, so reverting rather than leaving it broken-and-noisy is the right call. test-on-lab remains as a per-PR opt-in, so this doesn't remove the ability to test these repos, just the automatic (and currently guaranteed-failing) dispatch.


Generated by Claude Code

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