ci: publish build artifacts for test suite and loadlinux#68
Open
Brian Cain (androm3da) wants to merge 3 commits into
Open
ci: publish build artifacts for test suite and loadlinux#68Brian Cain (androm3da) wants to merge 3 commits into
Brian Cain (androm3da) wants to merge 3 commits into
Conversation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Brian Cain (androm3da)
force-pushed
the
ci/workflow-artifacts
branch
from
de7a32e to
6788ea5
Compare
Brian Cain (androm3da)
force-pushed
the
ci/workflow-artifacts
branch
2 times, most recently
from
c4c1cd7 to
627329c
Compare
Mutable tag references can be silently repointed by the action owner, enabling supply-chain attacks. Pin to the commit SHA each tag currently resolves to, keeping the tag name as a comment. - actions/checkout@v6 -> df4cb1c (v6.0.3) - actions/cache@v5 -> caa2961 (v5.1.0) Signed-off-by: Brian Cain <brian.cain@oss.qualcomm.com>
linux/makefile used -I$(KERNELPATH)/include which points to kernel/include in the source tree — a path that does not exist. Kernel-internal headers (context.h, cpuint.h, shint.h, linear.h, max.h, etc.) are generated into the build tree during the kernel build, not the source tree. Derive KERNEL_BUILD_DIR from INSTALLPATH the same way Makefile.inc.test does: KERNEL_BUILD_DIR := $(patsubst %/install,%/build/kernel,$(INSTALLPATH)) and use -I$(KERNEL_BUILD_DIR)/include in CFLAGS. Signed-off-by: Brian Cain <brian.cain@oss.qualcomm.com>
Add artifact upload steps to the Build and Test workflow: - Upload test results (HTML report, JSON, test.out) after testall, with if: always() so failures are still captured. - Build loadlinux and package it with all test ELFs, bootvm_image files, booter binaries, and simulator config files (timer_v*.cfg, devsim_v*.cfg) into a single test-suite.tar.xz archive. The tarball can be extracted and used directly by hexagon-sim, archsim, or any other Hexagon simulator to re-run the test suite independently of the CI build environment. - Also publish loadlinux as a standalone artifact for convenience. - Pass explicit target name to make -C linux to avoid the default target being taken from an included makefile fragment. Signed-off-by: Brian Cain <brian.cain@oss.qualcomm.com>
Brian Cain (androm3da)
force-pushed
the
ci/workflow-artifacts
branch
from
627329c to
79162a7
Compare
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
2 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.
Add artifact upload steps to the Build and Test workflow: