Skip to content

ci: add builder and prebuilt-runtime Dockerfiles for PR CI - #27181

Merged
fengttt merged 2 commits into
matrixorigin:mainfrom
fengttt:ci-builder-dockerfiles
Aug 14, 2026
Merged

ci: add builder and prebuilt-runtime Dockerfiles for PR CI#27181
fengttt merged 2 commits into
matrixorigin:mainfrom
fengttt:ci-builder-dockerfiles

Conversation

@fengttt

@fengttt fengttt commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

What type of PR is this?

  • Test and CI

Which issue(s) this PR fixes:

issue #27076

What this PR does / why we need it:

Publish-prerequisite split from #27077, per the reviewer-required rollout order there. The nightly ci-builder job (already on matrixorigin/CI@main via matrixorigin/CI#418) checks out main and skips gracefully while optools/images/Dockerfile.ci-builder is absent — so the first builder image cannot publish until this file lands.

Zero behavior change for any existing job: nothing on main references either file until the entrypoint wiring in #27077 merges. What this unblocks, in order:

  1. Dispatch/nightly of "Build and Push Images" publishes matrixorigin/matrixone:ci-builder (amd64, Docker Hub + ACR mirror).
  2. The UT and coverage-UT cache seeds already live in matrixorigin/CI ci.yaml go from no-op to active (guarded, continue-on-error, cold-build fallback — validated in ci: shared per-run build + nightly-warmed builder image for PR CI CI#418's review rounds).
  3. The canary ci: wire shared build and nightly builder image into PR CI #27077's reviewers require (shared-build path incl. forced cached-build failure) becomes runnable, after which ci: wire shared build and nightly builder image into PR CI #27077 leaves draft.

Both Dockerfiles passed docker build --check, and a full local image build on current main + fingerprint/warm-status verification is documented in matrixorigin/CI#418.

🤖 Generated with Claude Code

Prerequisite of the PR-CI speedup (matrixorigin#27076): publishing the nightly
ci-builder image requires these files on main, because the nightly job
(matrixorigin/CI image-build.yaml, ci-builder job — already merged via
matrixorigin/CI#418) checks out main and skips gracefully while they
are absent.

Zero behavior change for any existing job: nothing on main references
either file until the entrypoint wiring in matrixorigin#27077 lands. Merging this
first lets the first image publish so the UT/coverage-UT cache seeds
(already live in matrixorigin/CI ci.yaml) activate, and enables the
canary run matrixorigin#27077's reviewers require before that PR leaves draft.

See optools/images/Dockerfile.ci-builder for the warm-flavor design and
Dockerfile.prebuilt for the runtime-image assembly contract.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@fengttt
fengttt requested a review from XuPeng-SH as a code owner August 14, 2026 18:01
@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@fengttt
fengttt merged commit 46be786 into matrixorigin:main Aug 14, 2026
0 of 3 checks passed
@matrix-meow matrix-meow added size/XS Denotes a PR that changes [1, 9] lines and removed size/M Denotes a PR that changes [100,499] lines labels Aug 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/test-ci size/XS Denotes a PR that changes [1, 9] lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants