Skip to content

ci: test the merged release commit in Standalone regression - #477

Merged
XuPeng-SH merged 1 commit into
mainfrom
codex/standalone-merged-subject-20261113
Nov 13, 2026
Merged

XuPeng-SH merged 1 commit into
mainfrom
codex/standalone-merged-subject-20261113

Conversation

@XuPeng-SH

@XuPeng-SH XuPeng-SH commented Nov 13, 2026 •

Copy link
Copy Markdown
Contributor

Standalone post-merge BVT runs through pull_request_target, but both MatrixOne checkouts omit ref and therefore select the default branch instead of the merged release commit.

Bind both checkouts to github.event.pull_request.merge_commit_sha. The existing merged-only job guards ensure the commit is already trusted base-repository code before opting into checkout v6's explicit-ref behavior. Add the same narrowly scoped actionlint compatibility exception already used by TKE for that supported checkout input.

Validation: hosted Check Yaml passed, including all workflow schema/lint checks and the existing contract suites. Local actionlint v1.7.12 passes; YAML semantic comparison confirms that both checkouts use the merged SHA and that existing guards, other checkouts and job behavior are retained. The checkout input was verified against actions/checkout@v6 metadata. Hosted BVT execution is not claimed: the MatrixOne Standalone caller is currently manually disabled.

Related to matrixorigin/matrixone#32713.

@XuPeng-SH
XuPeng-SH merged commit 5fc75d8 into main Nov 13, 2026
1 check passed
@XuPeng-SH
XuPeng-SH deleted the codex/standalone-merged-subject-20261113 branch November 13, 2026 15:02
XuPeng-SH added a commit to matrixorigin/matrixone that referenced this pull request Nov 13, 2026
## What type of PR is this?

- [ ] API-change
- [x] BUG
- [ ] Improvement
- [ ] Documentation
- [ ] Feature
- [x] Test and CI
- [ ] Code Refactoring

## Which issue(s) this PR fixes:

Fixes #32713

## What this PR does / why we need it:

Pull requests targeting `v4.2.8-hotfix` do not match the default-branch
entrypoint's numeric branch filter, so `MatrixOne ALL CI` never starts.
GitHub evaluates `pull_request_target` using the default branch;
changing the workflow only on the hotfix branch cannot fix admission.

Add the exact hotfix branch to the `main` entrypoint. This supersedes
#32714. The existing PR checks and `CI Required` result then apply on
the next eligible hotfix PR event.

Validation: actionlint v1.7.12 passes. YAML semantic comparison verifies
that the hotfix branch is the only added target and all other workflow
behavior is retained.

After merge, synchronize or reopen an existing hotfix PR to trigger
validation; earlier events are not replayed. The merged #32704 does not
receive CI retroactively.

Post-merge Standalone/TKE callers are manually disabled in GitHub, so
this PR makes no claim to restore those workflows. The separate
Standalone checkout correction is matrixorigin/CI#477.
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