ci: reduce load on pr runs - #1815
Merged
Ricardo Salveti (ricardosalveti) merged 2 commits intoMar 27, 2026
Merged
Ricardo Salveti (ricardosalveti) merged 2 commits into
Ricardo Salveti (ricardosalveti) merged 2 commits into
Conversation
Contributor
Author
|
Validating the push run via next: https://github.com/qualcomm-linux/meta-qcom/actions/runs/23570319988 |
Contributor
Author
|
Viswanath Kraleti (@vkraleti) while we have #1418 for introducing weekly, I wanted to first optimize our prs runs by moving a few to nightly, as the current pr load is way too much. |
Introduce a build profile input field to the reusable build-yocto workflow and use it to skip all debug and performance build combinations when performing CI on PRs. This reduces PR turnaround time while keeping full coverage on push and nightly. Signed-off-by: Ricardo Salveti <ricardo.salveti@oss.qualcomm.com>
Migrate most compile_warm_up tasks to the standard compile matrix as gnerating parallel cache with so many combinations provides little incremental cache benefit while consuming a substantial amount of CI resources. This change should also help reducing the initial load on jobs that are known to fail, since the current nodistro and qcom-distro combination is enough to cover most common problems. This change has no impact to the complete set of runs. Signed-off-by: Ricardo Salveti <ricardo.salveti@oss.qualcomm.com>
Ricardo Salveti (ricardosalveti)
force-pushed
the
nightly
branch
from
March 26, 2026 17:43
b309a8a to
d0f686a
Compare
| steps: | ||
| - uses: actions/checkout@v6 | ||
| - name: checkout | ||
| if: inputs.profile != 'pr' || (matrix.distro.name != 'debug' && matrix.distro.name != 'performance') |
Contributor
There was a problem hiding this comment.
can you not use github.event_name here , instead of making an additional inputs?
Contributor
Author
There was a problem hiding this comment.
For pr, yes, but I was thinking that having an extra argument makes it more explicit and also not bound to a potential event rename. We also expect to have a different permutation between nighly / weekly / monthly as well, so that could use the same input variable.
Test run workflowTest jobs for commit d0f686a
All jobs summary
|
Test Results 28 files 28 suites 1h 3m 11s ⏱️ For more details on these failures, see this check. Results for commit d0f686a. |
Jose Quaresma (quaresmajose)
approved these changes
Mar 27, 2026
Viswanath Kraleti (vkraleti)
approved these changes
Mar 27, 2026
Ricardo Salveti (ricardosalveti)
merged commit Mar 27, 2026
79f43d9
into
qualcomm-linux:master
485 of 677 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Reduce the load on our PR runs by reducing the amount of warm_up runs (for better cache usage) and skipping debug and performance builds.
Debug builds are only consumed from nightly runs and performance has no major difference from the normal build, so an extra run to catch possible issues there is not that useful in the end.