Skip to content

ci: add post-compile success job#1800

Merged
ricardosalveti merged 1 commit intoqualcomm-linux:masterfrom
quic-viskuma:master
Mar 26, 2026
Merged

ci: add post-compile success job#1800
ricardosalveti merged 1 commit intoqualcomm-linux:masterfrom
quic-viskuma:master

Conversation

@quic-viskuma
Copy link
Copy Markdown
Contributor

@quic-viskuma quic-viskuma commented Mar 23, 2026

The publish-summary checkrun was used by various automations to trigger
subsequent CI jobs on build success. This step was updated to always run
regardless of build compilation failures making it an unreliable trigger.
(e9542ed)

This change adds a success-only job after all compile matrix builds pass.
The resulting check can be reused by downstream workflows.

Fixes: #1796

Comment thread .github/workflows/build-yocto.yml Outdated
Copy link
Copy Markdown
Contributor

@lumag lumag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also please tune your commit message. Start by describing the issue you observe and the reasons for the change.

@ricardosalveti
Copy link
Copy Markdown
Contributor

You are describing what this PR is doing and not why it is needed, please explain that as part of your commit message.

@quic-viskuma
Copy link
Copy Markdown
Contributor Author

You are describing what this PR is doing and not why it is needed, please explain that as part of your commit message.

updated the commit message.

@quic-viskuma quic-viskuma force-pushed the master branch 2 times, most recently from e2011e4 to 942ea84 Compare March 24, 2026 05:39
@quic-viskuma quic-viskuma requested a review from lumag March 25, 2026 07:34
@ricardosalveti
Copy link
Copy Markdown
Contributor

Please rebase to fix the CI issues.

@ndechesne
Copy link
Copy Markdown
Contributor

We had to cancel the workflow/CI since we fixed a critical CI issue. Can you please rebase and push again?

The publish-summary checkrun was used by various automations to trigger
subsequent CI jobs on build success. This step was updated to always run
regardless of build compilation failures making it an unreliable trigger.
(qualcomm-linux@e9542ed)

This change adds a success-only job after all compile matrix builds pass.
The resulting check can be reused by downstream workflows.

Signed-off-by: Vishal Kumar <viskuma@qti.qualcomm.com>
@quic-viskuma
Copy link
Copy Markdown
Contributor Author

We had to cancel the workflow/CI since we fixed a critical CI issue. Can you please rebase and push again?

done

@ricardosalveti ricardosalveti merged commit eff4045 into qualcomm-linux:master Mar 26, 2026
6 of 38 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.

Publish summary executes even if builds are failing

4 participants