Skip to content

ci: fix test results not commenting on PRs (backport to 4.4.x)#2463

Merged
jbonofre merged 1 commit intoapache:karaf-4.4.xfrom
jbonofre:jbonofre/karaf-4.4.x-fix-test-results-comments
Mar 19, 2026
Merged

ci: fix test results not commenting on PRs (backport to 4.4.x)#2463
jbonofre merged 1 commit intoapache:karaf-4.4.xfrom
jbonofre:jbonofre/karaf-4.4.x-fix-test-results-comments

Conversation

@jbonofre
Copy link
Member

Summary

Backport of #2462 to karaf-4.4.x.

  • Upload the GitHub event payload file as an artifact in the CI workflow
  • Download it in the test-results workflow and pass it to publish-unit-test-result-action
  • Fix reference to non-existent github.event.workflow_run.event_path property

The publish-unit-test-result-action in workflow_run mode needs the original event file to identify the associated PR and post comments. Without it, the action creates check runs but cannot comment on PRs.

Test plan

  • Verify the CI workflow uploads the event-file artifact
  • Verify the test-results workflow downloads and uses the event file
  • Confirm PR comments appear after a test run completes

Backport of apache#2462 to karaf-4.4.x.

The publish-unit-test-result-action in workflow_run mode needs the
original event payload file to identify the associated PR. The CI
workflow was not uploading it, and the test-results workflow was
referencing a non-existent property (github.event.workflow_run.event_path).

Upload the event file as a separate artifact in CI and download it in
the test-results workflow so the action can correctly post PR comments.
@github-actions
Copy link

Test Results

242 files  ±0  242 suites  ±0   28m 30s ⏱️ +15s
993 tests ±0  950 ✅ ±0  43 💤 ±0  0 ❌ ±0 
997 runs  ±0  954 ✅ ±0  43 💤 ±0  0 ❌ ±0 

Results for commit a12918d. ± Comparison against base commit f89a316.

@jbonofre jbonofre merged commit 28fbd73 into apache:karaf-4.4.x Mar 19, 2026
3 checks passed
@jbonofre jbonofre deleted the jbonofre/karaf-4.4.x-fix-test-results-comments branch March 19, 2026 14:22
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