Skip to content

ci: gate the Go collector MVP path - #566

Merged
zzylol merged 4 commits into
mainfrom
ci/mvp-go-gate
Sep 2, 2026
Merged

zzylol merged 4 commits into
mainfrom
ci/mvp-go-gate

Conversation

@zzylol

@zzylol zzylol commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • test the repository-owned Go precompute and Gorilla contracts
  • test the asapedgeprocessor CollectorPlan/OpAMP runtime boundary
  • test the Python MVP harness contracts and validate the demo entrypoint
  • initialize the pinned Collector core and Go sketch dependencies required by those tests
  • rerun the gate when a stacked PR changes its base

Why

The existing MVP collector job only exercised asap-precompute-rs. The documented Collector MVP runtime is Go, while Rust is one precompute/contract implementation. This change makes the required check cover both sides without claiming that pre-merge tests replace the canonical four-node measured run.

Verification

  • workflow YAML parses successfully
  • git diff --check
  • GitHub Actions is the authoritative clean-checkout run for pinned submodules and private dependencies

MVP evidence boundary

This gate proves code-level contracts and harness behavior. The canonical acceptance evidence remains deploy/mvp-multinode/scripts/run_demo.sh all, which must produce MVP_RESULTS.json and MVP_REPORT.md.

@zzylol

zzylol commented Sep 2, 2026

Copy link
Copy Markdown
Contributor Author

The Go gate is intentionally red and has exposed a real clean-checkout blocker, not a CI-only issue:

This PR should not be weakened back to Rust-only coverage. The dependency stack needs to land (and ASAPCollector needs one consolidated pin) before this required gate can become green.

@zzylol
zzylol merged commit 162c43c into main Sep 2, 2026
1 check passed
@zzylol
zzylol deleted the ci/mvp-go-gate branch September 2, 2026 17:57
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