Skip to content

ci: split bazel-bootstrap diagnostics out of required CI Gate path #446

Description

@DecisionNerd

Summary

Required Bazel Bootstrap currently mixes authoritative gates (drift, ledger, //:ci_rust_tests, binding smokes) with diagnostic dual-build parity and cache observe/collect. Fabricated zero-hit JSON via || echo can mask harness failures. Checked-in perf-sample.json is treated as a live regression gate rather than one-shot M2 evidence.

Acceptance criteria

  • Required bazel-bootstrap keeps drift, ledger/parity unit tests, //:ci_rust_tests, and binding cdylibs/smokes
  • Dual-build parity + cache observe/collect move to non-required bazel-diagnostics job not in ci-gate needs
  • Remove fabricated zero-hit || echo JSON fallback; fail closed on harness crashes
  • Document perf-sample.json as one-shot M2 evidence (not a live evaluate regression gate)
  • CI Gate still requires the slim authoritative bazel-bootstrap result

Non-goals

  • Binding RC Bazel cutover (PR11)
  • Pre-push Bazel (PR8)

BDD

  • Given Bazel path is enabled, When dual-build parity or cache observe fails, Then CI Gate can still pass if authoritative bootstrap passed
  • Given cache log parse harness crashes, When diagnostics run, Then the job fails (no zero-hit fabricated JSON)

Source

Devinfra Audit Fix-It Plan Wave C / PR7

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions