Skip to content

ci: publish queued x86 E2E placeholders on PR checks - #7278

Merged
zhangzujian merged 1 commit into
kubeovn:masterfrom
zhangzujian:fix/e2e-pr-check-placeholders
Aug 20, 2026
Merged

ci: publish queued x86 E2E placeholders on PR checks#7278
zhangzujian merged 1 commit into
kubeovn:masterfrom
zhangzujian:fix/e2e-pr-check-placeholders

Conversation

@zhangzujian

Copy link
Copy Markdown
Member

The trusted executor reporter only mirrored GitHub Actions jobs that already existed in the current run. Selected E2E jobs needs: build-kube-ovn, so during image build selectedExecutorJobs() is empty and pull request HEAD checks stay blank. That is what happened on #7275: the pull_request reporter is skipped, and the trusted workflow_dispatch reporter had nothing to upsert yet.

This change publishes queued placeholders for every selected job as soon as the reporter starts, also mirrors the running infrastructure jobs (Build kube-ovn, Build E2E Binaries, Kind image prep, vpc-nat-gateway), and skips the generic matrix placeholders once the real (ipv4, overlay) cells exist.

Fixes the live PR-check gap for #7230. After merge, a new trusted executor (automatic or /test e2e ...) should show in-progress Build kube-ovn and queued conformance checks on the PR HEAD before E2E jobs are queued.

Signed-off-by: Zujian Zhang zhangzujian.7@gmail.com

The trusted executor reporter only mirrored jobs that already existed in
the Actions run. During Build kube-ovn the selected E2E jobs are not
queued yet, so pull request HEAD checks stayed empty.

Publish queued placeholders for selected jobs immediately, mirror the
running infrastructure jobs, and skip matrix placeholders once the real
matrix cells start.

Signed-off-by: Zujian Zhang <zhangzujian.7@gmail.com>
@zhangzujian
zhangzujian requested a review from oilbeater as a code owner August 20, 2026 09:24
@dosubot dosubot Bot added size:L This PR changes 100-499 lines, ignoring generated files. ci github_actions Pull requests that update GitHub Actions code labels Aug 20, 2026
@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 32353764986

Coverage increased (+0.004%) to 32.748%

Details

  • Coverage increased (+0.004%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 64588
Covered Lines: 21151
Line Coverage: 32.75%
Coverage Strength: 0.38 hits per line

💛 - Coveralls

@zhangzujian
zhangzujian merged commit e0d20f1 into kubeovn:master Aug 20, 2026
71 checks passed
@zhangzujian
zhangzujian deleted the fix/e2e-pr-check-placeholders branch August 20, 2026 10:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci github_actions Pull requests that update GitHub Actions code size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants