Skip to content

chore(pr-gate): add F8 allowlist annotation#18

Open
noahgift wants to merge 2 commits into
mainfrom
chore/falsify-f8-allow-pr-gate
Open

chore(pr-gate): add F8 allowlist annotation#18
noahgift wants to merge 2 commits into
mainfrom
chore/falsify-f8-allow-pr-gate

Conversation

@noahgift
Copy link
Copy Markdown
Contributor

Summary

  • Annotate the authorize job with `# falsify-f8-allow: ...` so the paiml/infra F8 build-performance gate records this as allowlisted rather than failing.
  • The PR authorization step legitimately must run before any self-hosted runner is claimed.

Test plan

  • `cargo run --example falsify_f8_no_github_hosted -- --json` shows 0 offenders, 3 allowlisted (incl. this file)
  • CI green on this PR

Refs paiml/infra build-performance.md F8 gate

🤖 Generated with Claude Code

noahgift and others added 2 commits April 18, 2026 15:03
…fs paiml/infra#PMAT-156)

Build-performance spec Phase 1 (YAML-only, safe changes):

- Add CARGO_BUILD_JOBS=4 default to top-level env. Prevents cargo
  oversubscription when many runners share a host.
  Spec: build-performance.md §4.1 (runners × jobs ≤ 1.2 × ncores).

- Migrate provenance job from ubuntu-latest to [self-hosted, clean-room].
  Spec principle §3.8: no GitHub-hosted runners.

Follow-ups (not in this PR): lint/test/build label splitting (§7.2),
nextest migration (Phase 2).
The F8 build-performance gate forbids GitHub-hosted runners for
steady-state CI, but the PR authorization step legitimately must
run *before* a self-hosted runner is claimed — otherwise untrusted
contributors could trigger self-hosted workloads. Annotate with the
standard ` + '`# falsify-f8-allow:`' + ` comment so the gate records this
as allowlisted rather than failing.

Refs paiml/infra (build-performance.md F8)

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@noahgift noahgift enabled auto-merge (squash) April 18, 2026 14:15
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