Skip to content

fix(ci): accept isolated executor refs in x86 E2E selection - #7254

Merged
zhangzujian merged 1 commit into
kubeovn:masterfrom
zhangzujian:fix/isolated-executor-ref-name
Aug 19, 2026
Merged

fix(ci): accept isolated executor refs in x86 E2E selection#7254
zhangzujian merged 1 commit into
kubeovn:masterfrom
zhangzujian:fix/isolated-executor-ref-name

Conversation

@zhangzujian

Copy link
Copy Markdown
Member

What this PR does / why we need it

#7253 unblocked creating the disposable automatic executor ref. The
executor then failed immediately in x86 E2E Selection:

executor must run from the pull request base branch

Selection compared GITHUB_REF_NAME to pull_request.base.ref. Isolated
executor refs are named x86-e2e/pr-<n>-a-<approval>-d-<generation> and
only point at the base SHA, so that check can never succeed for the
intended dispatch path.

This keeps the SHA == base check, and accepts either the real base branch
or the exact isolated executor ref for that approval/dispatch generation.

Evidence from the first post-hotfix executor:

Until this lands on master, workflow_dispatch still loads the old
selector from the isolated ref, so this PR's own automatic executor will
keep failing that check.

Which issue(s) this PR fixes

Follow-up to #7253 / #7244. Related to #7230.

Special notes for reviewers

hack/test_e2e_control.py now covers the exact isolated ref from PR #7251
generation 32235889723.

Automatic dispatch creates a disposable x86-e2e/pr-* ref that points at
the PR base SHA. Selection compared GITHUB_REF_NAME to the base branch
name, so every isolated executor failed before running smoke.

Signed-off-by: Zujian Zhang <zhangzujian.7@gmail.com>
@zhangzujian
zhangzujian requested a review from oilbeater as a code owner August 19, 2026 09:13
@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. ci labels Aug 19, 2026
@zhangzujian
zhangzujian merged commit 78363e0 into kubeovn:master Aug 19, 2026
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant