CI: Skip jobs arm-08 to arm-14 when a Complex PR is created / updated#14386
Merged
Conversation
When we submit or update a Complex PR that affects All Architectures (Arm, RISC-V, Xtensa, etc): CI Workflow shall run only half the jobs. Previously CI Workflow will run `arm-01` to `arm-14`, now we will run only `arm-01` to `arm-07`. When the Complex PR is Merged: CI Workflow will still run all jobs `arm-01` to `arm-14` Simple PRs with One Single Arch / Board will build the same way as before: `arm-01` to `arm-14` This is explained here: apache#14376 Note that this version of `arch.yml` has diverged from `nuttx-apps`, since we are unable to merge apache#14377
github-actions
Bot
added
Area: CI
Size: XS
anchao
approved these changes
Oct 17, 2024
Member
Author
|
Thanks @anchao for reviewing my PR! Right now it's a bit messy because this PR conflicts with #14377. So we have to decide which PR to merge first, then I'll update the second PR. Really sorry for the rush, we are trying to cut down GitHub Costs before ASF shuts down our servers in 2 weeks. Thanks :-) |
cederom
approved these changes
Oct 18, 2024
cederom
approved these changes
Oct 18, 2024
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
3 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.
Summary
When we submit or update a Complex PR that affects All Architectures (Arm, RISC-V, Xtensa, etc): CI Workflow shall run only half the jobs. Previously CI Workflow will run
arm-01toarm-14, now we will run onlyarm-01toarm-07.When the Complex PR is Merged: CI Workflow will still run all jobs
arm-01toarm-14Simple PRs with One Single Arch / Board will build the same way as before:
arm-01toarm-14This is explained here: #14376
Note that this version of
arch.ymlhas diverged fromnuttx-apps, since we are unable to merge #14377Impact
When we submit or update a Complex PR: CI Workflow shall run only half the jobs:
arm-01toarm-07(instead ofarm-01toarm-14)No changes to the CI Workflow when we merge a Complex PR. No impact for Simple PRs.
Testing
Creating a Simple PR will run jobs
arm-01toarm-14(like before)https://github.com/lupyuen5/label-nuttx/actions/runs/11380165777
Creating a Complex PR will only run jobs
arm-01toarm-07(skippingarm-08toarm-14)https://github.com/lupyuen5/label-nuttx/actions/runs/11380214455
Merging a Complex PR will run all jobs
arm-01toarm-14(like before)https://github.com/lupyuen5/label-nuttx/actions/runs/11380246952