Skip to content

Add commit status support to QEMU PR Validation#110

Open
makubacki wants to merge 1 commit intoOpenDevicePartnership:mainfrom
makubacki:add_pr_qemu_validation_status_check_support
Open

Add commit status support to QEMU PR Validation#110
makubacki wants to merge 1 commit intoOpenDevicePartnership:mainfrom
makubacki:add_pr_qemu_validation_status_check_support

Conversation

@makubacki
Copy link
Collaborator

@makubacki makubacki commented Mar 19, 2026

Closes #108

Adds an optional head-sha input to the three reusable workflow files. When provided, commit statuses are created on the PR head SHA so that QEMU validation appears in the PR status checks area.

This is needed because the validation chain uses workflow_run triggers, which run in the default branch context and are not automatically associated with the PR as status checks.

The status context is Patina QEMU PR Validation which can be configured as a required status check in branch protection rules.


Tested in a PR on my patina-dxe-core-qemu fork

Pending Results

image

Successful Result Status

image

Failure Result Status

image

@makubacki makubacki self-assigned this Mar 19, 2026
Adds an optional `head-sha` input to the three reusable workflow
files. When provided, commit statuses are created on the PR head SHA
so that QEMU validation appears in the PR status checks area.

This is needed because the validation chain uses `workflow_run`
triggers, which run in the default branch context and are not
automatically associated with the PR as status checks.

The status context is `Patina QEMU PR Validation` which can be
configured as a required status check in branch protection rules.

Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com>
@makubacki makubacki force-pushed the add_pr_qemu_validation_status_check_support branch from 755ca97 to b188442 Compare March 20, 2026 18:14
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.

[Feature]: Display Patina QEMU PR Validation status in PR status checks

3 participants