Skip to content

ci: run complementary BVT group on multi-CN compose - #419

Merged
XuPeng-SH merged 2 commits into
mainfrom
ci/multi-cn-complementary-bvt
Aug 13, 2026
Merged

ci: run complementary BVT group on multi-CN compose#419
XuPeng-SH merged 2 commits into
mainfrom
ci/multi-cn-complementary-bvt

Conversation

@XuPeng-SH

Copy link
Copy Markdown
Contributor

What this PR does

Replaces the active native LAUNCH, PESSIMISTIC BVT deployment with the same multi-CN Docker Compose topology already used by the complementary Proxy BVT job.

The two active BVT jobs now differ only in their complementary case-group assignment and the case-driven resource argument:

  • Compose Proxy runs compose_group.
  • Compose Pessimistic runs the complementary launch_group.

The existing generation/group manifests and distinct coverage artifact names are preserved, while the coverage merge contract now identifies the second deployment as compose-pessimistic.

Why

In matrixorigin/matrixone#27077 run 31663214904, the multi-CN Compose job completed successfully in about 42 minutes, while the native Launch job spent the full 75-minute job budget in Build MatrixOne and was cancelled before executing any BVT case. Reusing the proven Compose path removes that native cold-build bottleneck.

Lifecycle and failure handling

  • Multi-CN startup is bounded to 10 minutes.
  • Container and volume cleanup runs under always() independently of BVT and coverage results.
  • Coverage is generated after graceful Compose shutdown so instrumented processes flush GOCOVERDIR.
  • Existing logs, stack dumps, disk diagnostics, and failure artifacts are retained for the Compose deployment.

Validation

  • actionlint on both changed workflows
  • Ruby YAML parse on both changed workflows
  • simulated complementary group=0/1 manifests through the coverage merge jq predicate
  • git diff --check

Related: matrixorigin/matrixone#27077

@XuPeng-SH
XuPeng-SH merged commit 993a5fa into main Aug 13, 2026
1 check passed
@XuPeng-SH
XuPeng-SH deleted the ci/multi-cn-complementary-bvt branch August 13, 2026 04:57
fengttt added a commit that referenced this pull request Aug 14, 2026
… compose-based pessimistic BVT

Upstream (#419-#423) landed two things this branch must integrate with:

- The SCA job gained its own trusted actions/cache pipeline (module
  cache from matrixorigin/matrixone's sca-go-module-cache.yaml producer,
  native prerequisites keyed on compiler identity plus full native-input
  hashes, cached static-check tools). That mechanism subsumes this
  branch's SCA seed step — including the toolchain-ABI fingerprint this
  branch deliberately deferred — so the SCA seed is dropped, the
  nightly ci-builder publishes amd64 only (the arm64 leg existed solely
  for SCA), and the multi-arch manifest job goes away.

- The pessimistic standalone BVT was rebuilt on the compose topology
  (its native launch build kept exceeding the job timeout). The old
  native-binary artifact consumer no longer applies; both BVT jobs now
  consume the shared build identically, assembling the runtime image
  from the artifact via Dockerfile.prebuilt with the same guarded
  fallback to the full in-docker build.

The UT job keeps both the upstream Shanghai-proxy canary and this
branch's cache seed; they are independent.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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