Skip to content

ci: post check run for release S3 upload - #226

Merged
Keerthi Gowda (keerthi-go) merged 1 commit into
mainfrom
feat/release-checkrun
Aug 20, 2026
Merged

Keerthi Gowda (keerthi-go) merged 1 commit into
mainfrom
feat/release-checkrun

Conversation

@keerthi-go

@keerthi-go Keerthi Gowda (keerthi-go) commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add a check run (pkg-release-check-run) to the upload-debs-to-s3 job, posted as completed with the upload outcome and the S3 destination path, so Coral can subscribe to the check_run event instead of the Status API.
  • Write the check run's html_url to GITHUB_STEP_SUMMARY.
  • Grant checks: write in the pkg-release.yml stub under .github/pkg-workflows/qli-ci/, since that permission must be declared by the caller workflow for the reusable workflow's check-run step to run. This stub is synced to all pkg-* repos.

Comment thread .github/workflows/pkg-release-reusable-workflow.yml Fixed
@keerthi-go
Keerthi Gowda (keerthi-go) force-pushed the feat/release-checkrun branch 2 times, most recently from 5165b4d to 9cd1a30 Compare August 19, 2026 22:42
Add a check run to the upload-debs-to-s3 job so Coral can subscribe to
its check_run event instead of the Status API. The check run is named
pkg-release-check-run, posted once as completed with the upload
outcome, and includes the S3 destination path in its output so the
uploaded artifacts are traceable. Its html_url is also written to
GITHUB_STEP_SUMMARY.

Grant checks: write in the pkg-release.yml stub under
.github/pkg-workflows/qli-ci/, since that permission has to be
declared by the top-level caller workflow for the reusable workflow's
check-run step to run. This stub is synced out to all pkg-* repos, so
it covers every repo using the release workflow.

Signed-off-by: Keerthi Gowda <kbalehal@qti.qualcomm.com>
@keerthi-go
Keerthi Gowda (keerthi-go) merged commit c74c53f into main Aug 20, 2026
12 checks passed
@simonbeaudoin0935
Simon Beaudoin (simonbeaudoin0935) deleted the feat/release-checkrun branch August 31, 2026 18:34
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.

3 participants