Skip to content

ops(queue): enroll mhtml-etl-gateway in queue-health evidence - #2212

Closed
seonghobae wants to merge 2 commits into
mainfrom
codex/mhtml-queue-health-enrollment
Closed

seonghobae wants to merge 2 commits into
mainfrom
codex/mhtml-queue-health-enrollment

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

Parent owner: #1150. Downstream scientific consumer: ContextualWisdomLab/TEPP#536. Canary: ContextualWisdomLab/mhtml-etl-gateway#95.

Finding

ContextualWisdomLab/mhtml-etl-gateway#95 independently reproduces the organization Actions admission/runner queue class while canonical queue-health owner #1150 cannot observe that repository because its explicit bounded allowlist omits it.

Fresh exact canary is mhtml-etl-gateway#95@488ce69863148edd9bafdc54c40a2b2df1c4a8ef. Repository Quality run 34957509223 materializes Python 3.11–3.14 jobs, but all four remain queued with runner_id=0, empty runner name, and steps=[]. Exact-head Security Scan 34957509235, SAST Semgrep 34957509185, and CodeQL PR 34957509212 are also queued. TEPP #527 separately reproduces pre-checkout queueing on exact 8b5462dd547773688eed282a3ff373247778f4a1.

This is owner-side admission evidence, not permission for leaf reruns, no-op commits, runner-selector churn, cancellation, predecessor-status transfer or gate weakening.

RED → minimal repair

Parent exact authority is #1150@42bb922f03bf75aed1bc1931d9fbaf04a5433e20.

RED ce1c60bb1db1edaec71b55986ba771baae5c303a enrolls ContextualWisdomLab/mhtml-etl-gateway in config/actions_queue_health_repositories.json while intentionally leaving the exact-equality owner contract unchanged. On an executing runner, existing test_queue_health_allowlist_is_explicit_and_bounded must reject that mismatch.

Causal fix 22d6a9679f015cd2571ae859d1925d92c3e0d676 updates only tests/test_actions_queue_health_contract.py to admit the reviewed repository in the same explicit bounded list.

Effective delta against the parent is exactly two owner paths:

  • config/actions_queue_health_repositories.json
  • tests/test_actions_queue_health_contract.py

Collector logic, workflow triggers, runner selectors, cancellation/rerun behavior, required gates and observed leaf repositories are unchanged. The two commits were created as ordinary parent→RED→fix history and only the final fix head was published as the branch, avoiding a redundant intermediate hosted run.

Promotion boundary

Keep Draft. Enrollment is not incident resolution and no predecessor result transfers. GREEN requires canonical #1150 to normally integrate this two-path delta and the existing read-only collector to observe mhtml-etl-gateway using current repository/PR/head/workflow/job identities. Exact-head checks and review on this child remain authoritative for this child only.

No self-approval, administrator bypass, force update, destructive rebase, workflow-source copy, cancellation, blind rerun, no-op freshness commit, runner-selector churn, synthetic status or gate weakening is authorized.

Enroll mhtml-etl-gateway in the bounded queue-health configuration without yet updating the exact-equality contract test. This preserves a realistic RED in ordinary history: the existing allowlist contract must fail until the owner test explicitly accepts the new reviewed repository.

Refs #1150 ContextualWisdomLab/mhtml-etl-gateway#95
Update the exact-equality queue-health allowlist contract to include ContextualWisdomLab/mhtml-etl-gateway. Collector logic, workflow triggers, runner selectors, cancellation/rerun behavior and required gates remain unchanged.

Refs #1150 ContextualWisdomLab/mhtml-etl-gateway#95
@coderabbitai

coderabbitai Bot commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

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

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.

@seonghobae

Copy link
Copy Markdown
Contributor Author

Supersession note: unique mhtml enrollment delta carried to successor #2243 on current main (head 30a5e73e6418e2e9e655b0ff4c30b11996333612). This draft remains open pending verified carryover after #2243 merges; do not close on redundancy alone.

@seonghobae

Copy link
Copy Markdown
Contributor Author

Superseded — closing (tree-verified)

This draft's unique valid delta was enroll ContextualWisdomLab/mhtml-etl-gateway into config/actions_queue_health_repositories.json and the matching allowlist assertion in tests/test_actions_queue_health_contract.py.

Carryover proof on origin/main (not ancestry):

git grep -lF "ContextualWisdomLab/mhtml-etl-gateway" origin/main -- config/actions_queue_health_repositories.json
# → origin/main:config/actions_queue_health_repositories.json

git grep -lF "ContextualWisdomLab/mhtml-etl-gateway" origin/main -- tests/test_actions_queue_health_contract.py
# → origin/main:tests/test_actions_queue_health_contract.py

Successor merge: #2243

Three-dot note: git diff --stat origin/main...22d6a9679f015cd2571ae859d1925d92c3e0d676 is still non-empty because this branch forked from a pre-enrollment allowlist tip; the three-dot patch only re-adds strings already present on main (plus trailing-newline noise). Redundancy alone is not the close reason — content proof above is.

Closing per AGENTS.md "Verifying a superseded — closing claim."

@seonghobae

Copy link
Copy Markdown
Contributor Author

Closed as superseded: enrollment string ContextualWisdomLab/mhtml-etl-gateway is on origin/main via #2243.

@seonghobae seonghobae closed this Sep 17, 2026
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