Skip to content

ci: install GitHub CLI for distro validation - #84

Merged
Keerthi Gowda (keerthi-go) merged 1 commit into
mainfrom
fix/install-gh-cli-for-distro-validation
Aug 21, 2026
Merged

ci: install GitHub CLI for distro validation#84
Keerthi Gowda (keerthi-go) merged 1 commit into
mainfrom
fix/install-gh-cli-for-distro-validation

Conversation

@bjordiscollaku

Copy link
Copy Markdown
Contributor

Summary

  • Install GitHub CLI through APT in every trusted distro validation job before its scripts call the GitHub API.
  • Restore kernel context resolution, Check Run creation and completion, distro dispatch, failure reporting, and callback validation.
  • Remove reliance on the ephemeral runner image providing gh.

Root Cause

Smoke test PR #82 completed its kernel build and uploaded Debian packages successfully. The trusted workflow_run then failed in resolve-distro-validation-context.sh because gh was not installed, so no Check Run was created and no distro validation request was dispatched.

Validation

  • Parsed the workflow as YAML.
  • Validated the workflow with Actionlint and ShellCheck.
  • Confirmed all five jobs that invoke GitHub API helpers install gh first.
  • Confirmed Ubuntu provides the required ARM64 gh package.
  • Confirmed the workflow diff is clean and limited to the trusted validation workflow.

Counterpart distro fix: https://github.com/qualcomm-linux/qcom-distro-images/pull/127

The complete live request, distro build, callback, and Check Run chain will be rerun after both fixes are available on main.

- Install GitHub CLI from Ubuntu repositories in every trusted job that invokes gh api.
- Restore kernel context resolution, validation dispatch, failure reporting, callback validation, and Check Run updates.
- Remove reliance on the ephemeral runner image providing the GitHub CLI.

Signed-off-by: Bjordis Collaku <bcollaku@qti.qualcomm.com>
@keerthi-go
Keerthi Gowda (keerthi-go) merged commit 6d53414 into main Aug 21, 2026
21 checks passed
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.

2 participants