Skip to content

fix(ci): route immutable thread-global codegen tests - #11767

Closed
proggeramlug wants to merge 2 commits into
mainfrom
codex/thread-global-e2e-routing-20261002
Closed

proggeramlug wants to merge 2 commits into
mainfrom
codex/thread-global-e2e-routing-20261002

Conversation

@proggeramlug

@proggeramlug proggeramlug commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Integrated into draft #11756 at 79affe7b50bb109ce3f1d69e2754d7faebc5be72, alongside the gap-shard timeout repairs. Keep this PR open until the combined replacement is verified merged. Neither PR is currently merge-approved; final compiler/product/lint and hosted acceptance remain pending.

The e2e-scoped job rejects every core PR after #11734 added thread_immutable_globals without registering it in the codegen source-to-suite map. Register the existing three-test, in-process IR suite so it runs for codegen source changes.

Validation: the current-main planner self-test reproduces the unclassified-suite failure; the repaired self-test passes. Routing controls confirm direct suite selection, preserved existing mapped suites, the 300-second mapped timeout, coverage with --cap 1, and unchanged unrelated HIR scope. No suite is excluded.

Draft pending final compiler product/lint validation and hosted CI. This changes only test routing; the existing Rust suite and production code are unchanged.

@coderabbitai

coderabbitai Bot commented Oct 2, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@proggeramlug

Copy link
Copy Markdown
Contributor Author

Superseded: thread_immutable_globals is already mapped in scripts/ci_e2e_scope.py on main (#11768).

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.

1 participant