Repository navigation
ci(grc): stage central self-hosted Product producer - #2602
seonghobae wants to merge 7 commits into
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
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. Comment |
Current authority — exact refreshed stack
This COMMENT is evidence, not approval. No bypass, auto-merge, merge, runner relabeling, ACL expansion, or source-neutral event is authorized. |
Current authority — exact head
|
Combine the reviewed GRC central producer/caller split with the Noema current-producer evidence repair. Parents: - PR2602 GRC head 0cc1399 - PR2622 Noema repair 5c1196a Local source evidence: - tree a5fa1cf - Python 3.14 focused tests 532 passed, 2 skipped, 45 subtests - py_compile, actionlint, diff --check exit 0 - bounded Noema stale-evidence re-review APPROVE This remains Draft/proposed only. It is not formal App approval, runner activation, or protected merge.
Current-main refresh, exact head
|
Scope
Stage the GRC-owned central
workflow_callProduct producer on the canonical runner-routing owner #2565. The producer keeps fixed GRC caller admission, same-repository PR policy, dedicated self-hosted group/labels, and no hosted fallback or inherited secrets.The GRC delta remains four additive paths:
.github/workflows/grc-product.ymldocs/doctoring/grc-product-central-execution.mddocs/templates/grc-product.yml.intests/test_grc_product_reusable_workflow_contract.pyThe comparison also exposes #2609's one prerequisite-owned clock fixture. No implementation from either owner is copied.
Current exact-head topology — 2026-10-09
ab4a39d818a1d865852e25a97b4200e08ece0bbb056e49de04c025e2b48ca54c5b53cc11e2612a69bb5b297be7b99f584ddf0d587e595e3bbdf0fac2, parents prior GRC head3f832395800efac06690616b322443ca96e55109and current runner-policy owner ci: stage all workflows on isolated self-hosted runners #2565a206770680895adba000d7538fc3d5a5b499149ebb5b297be7b99f584ddf0d587e595e3bbdf0fac2and current-time fixture owner test(codeql): keep current-time fixtures fresh #26098ba032a3df47c8f401a7017ae84302c725c2f9c8ci/sdp-all-self-hosted-20261003@a206770680895adba000d7538fc3d5a5b499149eRoot cause and repair
The prior GRC head was still based on #2565 predecessor
6e924f32a2dbaa0950709c6d8f1391fc1058ef43, while canonical #2565 had advanced toa206770…. The two heads had diverged and the GRC body named stale owner evidence.The first ordinary merge refreshes the stack without changing the four GRC files. #2609 is added as a second prerequisite because its test fixture is required for a truthful current-date full-suite result. Runner-routing conflicts introduced through #2609's protected-main ancestry were resolved at the single writer: #2565's
CWL CI isolated+cwlab-ci-isolatedcontracts remain byte-equivalent to the owner.Exact-tree verification
GITHUB_ACTIONS=truegit diff --check, and clean worktree: GREENActivation and publication HOLD
Draft / Proposed / merge HOLD. Draft is required because mutable runner-policy prerequisite #2565 is not runtime-ready: isolated capacity, cleanup canary, terminal exact-head Checks, and qualifying approval remain absent.
orchestrator/free.