Skip to content

fix(ci): let the x86 E2E reducer post PR comments - #7270

Merged
zhangzujian merged 2 commits into
kubeovn:masterfrom
zhangzujian:fix/e2e-gate-pr-comment-write
Aug 20, 2026
Merged

fix(ci): let the x86 E2E reducer post PR comments#7270
zhangzujian merged 2 commits into
kubeovn:masterfrom
zhangzujian:fix/e2e-gate-pr-comment-write

Conversation

@zhangzujian

Copy link
Copy Markdown
Member

Follow-up to #7268. The reducer job still 403s when commenting that the isolated executor was not visible.

Comment-gated dispatch accepted /test e2e core, but the gate mutation job
could not record a durable approval intent. Creating the PR comment 403ed
because the job only had pull-requests: read.

Signed-off-by: Zujian Zhang <zhangzujian.7@gmail.com>
The approved-path reducer still had pull-requests: read, so it 403ed
when the isolated executor run was not visible and could not explain
the failure on the PR.

Signed-off-by: Zujian Zhang <zhangzujian.7@gmail.com>
@zhangzujian
zhangzujian requested a review from oilbeater as a code owner August 20, 2026 03:07
@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. ci github_actions Pull requests that update GitHub Actions code labels Aug 20, 2026
@zhangzujian
zhangzujian merged commit 4081927 into kubeovn:master Aug 20, 2026
35 checks passed
@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 32327131903

Warning

Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes.
Quick fix: rebase this PR. Learn more →

Coverage decreased (-0.003%) to 32.748%

Details

  • Coverage decreased (-0.003%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 2 coverage regressions across 1 file.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

2 previously-covered lines in 1 file lost coverage.

File Lines Losing Coverage Coverage
pkg/ovs/ovn-nb-bfd.go 2 61.61%

Coverage Stats

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

💛 - Coveralls

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:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants