Repository navigation
ci: migrate x86 UT coverage to CloudSigma runners - #475
Open
Ariznawlll wants to merge 17 commits into
Open
Ariznawlll wants to merge 17 commits into
Ariznawlll wants to merge 17 commits into
Conversation
Ariznawlll
marked this pull request as ready for review
November 11, 2026 09:24
Ariznawlll
marked this pull request as draft
November 11, 2026 20:07
Collaborator
Author
|
已补充 CS S3 接入合约测试和 JED 内部 endpoint 部署说明(a414b3e)。CS 路径按 runner 环境选择对应 secret;受信任 PR 配置不全时明确失败;真实 COS/OSS 不注入 CS 日常 UT;强制核对 CS S3 basic 和 streaming_write 实际 pass。修复 secret-free local MinIO canary 仍错误要求云凭证的检查。本地 13 项 Python 测试和 actionlint 通过;最新 Linux 合约及隔离读写清理验证运行于 https://github.com/matrixorigin/matrixone/actions/runs/43149448772 。依赖 matrixorigin/matrixone#32590;真实 S3 配置和验证未完成,保持 draft,不切换 main 或现有正式测试。 |
Ariznawlll
marked this pull request as ready for review
November 13, 2026 12:47
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Scope
Replace x86 UT Coverage's TKE runner with the shared
cs-large-7c14gpool. No manual environment selection: the actual runner selects MTY, MNL2, JED or RUH S3, and the whole Coverage job stays on that runner.CS_S3_*_CONFIGsecrets before enabling the pool. Unknown runners or missing matching configuration fail; no cross-environment fallback.safe to testtrust checks, package parallelism 8, package/suite deadlines and coverage reporting.ci-ut/<run-id>-<attempt>/isolation. Requires MatrixOne #32590, whose fixtures clean only their own objects.Simplification
Removed fixed-PR/SHA canaries, test-branch bypasses, temporary runner/parallelism inputs, local MinIO build/experiment logic and legacy cloud credential branches. Shortened documentation and retained only production-path contract tests. Existing unrelated cache inputs remain unchanged.
Validation
git diff --checkpassed.8ceb5f6: workflow schema/actionlint, native runtime, CS routing/isolation, mandatory S3 execution and skip comparison checks.Rollout gate — keep draft
Merge the MO fixtures first and finish same-source full Coverage / added-skip acceptance in all four environments before switching production. This cleanup does not merge main, change runner capacity, restart existing tests or touch stored data.