Skip to content

ci(pg-erd): centralize quality on isolated self-hosted runners - #2606

Draft
seonghobae wants to merge 6 commits into
fix/sdp-ghas-analyses-pagination-20261002from
ci/pg-erd-central-quality-20261008
Draft

seonghobae wants to merge 6 commits into
fix/sdp-ghas-analyses-pagination-20261002from
ci/pg-erd-central-quality-20261008

Conversation

@seonghobae

@seonghobae seonghobae commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Adds the fixed-consumer pg-erd-cloud quality reusable workflow to the organization control plane. The product-specific workflow retains backend/frontend gates and inherits the self-hosted-only CWL CI isolated + cwlab-ci-isolated routing contract from canonical runner-policy owner #2565. No hosted fallback, arbitrary executable input, inherited secrets, or author self-approval is added.

Current exact-head topology — 2026-10-09

Root cause and repair

The prior PR targeted main while claiming #2565's runner labels and custom actionlint vocabulary. That made the consumer a parallel writer of a mutable shared policy. Ordinary integration exposed seven true conflicts between protected-main runner routing and #2565. They were resolved identically to the already-published #2607 stack: #2565 remains the canonical owner of CWL CI isolated routing.

The first merged-tree repository suite then reproduced two time-dependent failures in tests/test_audit_org_codeql_coverage.py (5,512 passed, 2 failed, 10 skipped, 40 subtests). Those are the exact RED failures owned and repaired by #2609, so its exact head was integrated as a second merge parent instead of copying the fix.

Exact-tree executable evidence

  • Combined pg-erd, runner-policy, queue, and CodeQL-time contracts: 346 passed normally and 346 passed with GITHUB_ACTIONS=true
  • Warning-fatal complete repository suite: 5,514 passed, 10 skipped, 40 subtests
  • 39/39 workflow YAML files parse successfully
  • Python compileall, git diff --check, and clean worktree: GREEN
  • Current exact-head CodeQL is in progress; JavaScript Coverage and Repository Metadata are queued. Pending Checks are not source failures and were not rerun or replaced.

Activation requirements / lifecycle

Draft / Proposed / merge HOLD. Draft is required because #2565 is a mutable, unready runtime prerequisite whose isolated runner group/capacity and cleanup canary remain absent.

  • ci: stage all workflows on isolated self-hosted runners #2565 owns global routing, group/label policy, and isolated-capacity activation.
  • test(codeql): keep current-time fixtures fresh #2609 remains an independent Ready prerequisite and requires terminal exact-head Checks plus qualifying approval before merge.
  • Operator lane ContextualWisdomLab/linux-cluster-ops#326 owns disposable runner isolation/canary. Do not borrow or relabel privileged runners.
  • A thin consumer caller must use a real immutable central commit SHA without secrets: inherit; required check names must be reconciled without weakening protection.
  • Noema/OpenCode routes remain orchestrator/free; no provider/model/paid fallback is introduced.
  • No bypass, auto-merge, source-neutral rerun, predecessor retirement, or merge is authorized by this receipt.

Provenance correction retained

Earlier offline artifact manifests contained unsupported web-retrieval claims. They remain withdrawn. Local executable results and independently reviewed source bytes are evidence; live documentation retrieval must be verified before activation.

@coderabbitai

coderabbitai Bot commented Oct 9, 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
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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
seonghobae changed the base branch from main to ci/sdp-all-self-hosted-20261003 October 9, 2026 06:47

Copy link
Copy Markdown
Contributor Author

Current authority — exact stacked head

This COMMENT/evidence is not approval. No bypass, auto-merge, merge, runner relabeling, access expansion, or source-neutral rerun is authorized.

Copy link
Copy Markdown
Contributor Author

Current authority — exact head 56c1435ab16fec9ef196e3bd4773845acf8f50a2

This Draft stack now consumes #2565 causal owner repair 8d4a2eb1061c060360741d4015e4350ffde6a3eb as an ordinary merge parent.

  • exact tree: 18950f46bb7c6ad011dab2dd047fa7463ced29f7
  • base: ci/sdp-all-self-hosted-20261003 at that exact owner head
  • child delta remains bounded; the only inherited change is the fail-closed CWL law CI ↔ law-ai-agent-ci contract
  • affected contracts: 59 passed normally and 59 passed with GITHUB_ACTIONS=true
  • git diff --check: GREEN
  • Draft reason remains the mutable owner prerequisite; queued Checks or missing approval are merge blockers only
  • no force push, destructive rebase, rerun, or state toggle

… into ci/pg-erd-central-quality-20261008

# Conflicts:
#	tests/test_audit_org_codeql_coverage.py
Base automatically changed from ci/sdp-all-self-hosted-20261003 to fix/sdp-ghas-analyses-pagination-20261002 October 12, 2026 21:39

This branch has not been deployed

No deployments
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