ci: use latest ci-builds JSON as a reference#532
Draft
pillo79 wants to merge 4 commits into
Draft
Conversation
92337bb to
23b1d65
Compare
|
Memory usage change @ 23b1d65
Click for full report table
Click for full report CSV |
CORE_BRANCH is supposed to be the branch that triggered the workflow, not the branch with the PR changes (that one is not accessible from the ci-builds repo). When a PR is merged, the workflow is triggered on the target branch, so we should use base_ref instead of head_ref to get the correct branch also at PR time.
23b1d65 to
d2fd8af
Compare
|
Memory usage change @ d2fd8af
Click for full report table
Click for full report CSV |
f484be7 to
a0f67f1
Compare
|
Memory usage change @ a0f67f1
Click for full report table
Click for full report CSV |
08a0271 to
a5cb725
Compare
|
Memory usage change @ a5cb725
Click for full report table
Click for full report CSV |
fa292f8 to
f3e7db1
Compare
|
Memory usage change @ f3e7db1
Click for full report table
Click for full report CSV |
Prepare the build environment variables early in the workflow so that the lengthy Zephyr install process takes place after the variables are set up. This allows for easier quick inspection of the build state and better organization of the workflow.
On pull requests and pushes to the official ArduinoCore-zephyr repo, use the latest ci-builds JSON as a reference for building the core, if available. This allows to install the current set of tools required by the branch sources, instead of using those from the latest release. If the ci-builds JSON is not available for the current branch, the branch that is closest to the current one will be used instead, with a warning.
This commit adds a new job to the GitHub Actions workflow that prepares a temporary package index JSON for this pull request. The job merges the newly built core packages with the latest package index from ci-builds, allowing for offline testing and validation of the changes introduced in this PR. Note that using this currently requires a nightly build of the Arduino CLI that supports GitHub authentication and its associated auth setup.
f3e7db1 to
a3025c9
Compare
|
Memory usage change @ a3025c9
Click for full report table
Click for full report CSV |
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
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.
On pull requests and pushes to the official ArduinoCore-zephyr repo, use the latest ci-builds JSON as a reference for building the core, if available. This allows to install the current set of tools required by the branch sources, instead of using those from the latest release.
On pull requests to the official repo, the same logic is also used to compute a temporary package index that contains links to the PR's artifacts so that the CI build results can be directly installed locally and tested on an actual board. This feature requires an
arduino-cliwith cli: fix package index URLs not ending with a file name arduino-cli#3227 applied, which can be found in the CI-generated PR binaries page.