Skip to content

ci: return UT to single runner - #27823

Merged
XuPeng-SH merged 1 commit into
matrixorigin:mainfrom
XuPeng-SH:codex/revert-ut-sharding
Aug 28, 2026
Merged

ci: return UT to single runner#27823
XuPeng-SH merged 1 commit into
matrixorigin:mainfrom
XuPeng-SH:codex/revert-ut-sharding

Conversation

@XuPeng-SH

Copy link
Copy Markdown
Contributor

What changed

Disable the opt-in four-way race-UT matrix introduced by #27815 and restore the MatrixOne workflow caller to its previous single-runner configuration:

  • remove ut_sharded: true;
  • restore ut_parallel: 8 in the caller.

Resulting CI behavior

CI #438 remains backward compatible and defaults ut_sharded to false. Therefore this caller now expands to exactly one all matrix entry:

  • one UT Test on Ubuntu/x86 runner executes the complete race suite;
  • UT_SHARD=all retains every light, HNSW, issues, embedded, heavy, engine, and plan stage;
  • the shard summary job is skipped before runner allocation;
  • the established required-check name remains unchanged.

This intentionally does not fully revert #27815. Its runner-local improvements remain: deterministic HNSW coverage with less duplicate work, global-mock restoration, temporary-file cleanup, package partition validation, and safer package grouping. Only the resource-expensive four-runner activation is rolled back.

CI #438 does not need to be reverted: it is opt-in, defaults to the old one-runner behavior, and its immutable-head checkout and fail-closed sharding support are harmless while disabled.

Validation

  • the MatrixOne entrypoint is byte-for-byte identical to the parent of test: reduce race UT critical path #27815;
  • YAML syntax parse: pass;
  • git diff --check: pass;
  • final delivery diff: one workflow file, 1 insertion and 4 deletions.

@XuPeng-SH
XuPeng-SH requested a review from fengttt as a code owner August 28, 2026 22:52
@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 →

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

Labels

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