Skip to content

Route Codex runner diagnostics through approved qualification workflow - #24

Merged
KeyffMS merged 3 commits into
mainfrom
fix/runner-diagnostics-allowed-workflow
Sep 4, 2026
Merged

Route Codex runner diagnostics through approved qualification workflow#24
KeyffMS merged 3 commits into
mainfrom
fix/runner-diagnostics-allowed-workflow

Conversation

@KeyffMS

@KeyffMS KeyffMS commented Sep 4, 2026

Copy link
Copy Markdown
Owner

Summary

The standalone diagnostics workflow was rejected by the self-hosted runner administrator hook before any diagnostic step ran:

PlanAnvil runner policy denied this job: workflow file or workflow ref is not allowed

This change keeps the runner policy intact and routes diagnostics through the already-approved .github/workflows/plananvil-codex-qualification.yml workflow.

Changes

  • add diagnostics to the existing qualification workflow mode choices;
  • run the 14-variant diagnostic matrix only when mode=diagnostics;
  • keep diagnostics non-gating with respect to C01-C16 and the release gate;
  • remove the standalone diagnostics workflow that the runner allowlist rejects;
  • update regression tests to assert that diagnostics use the approved workflow file and remain main-only/self-hosted/sanitized.

Safety

No change to runner policy, no approval/sandbox bypass, no privilege expansion, and no weakening of the release gate.

@KeyffMS
KeyffMS merged commit 4fb7083 into main Sep 4, 2026
7 checks passed
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