Skip to content

Commit 969afcb

Browse files
Fixes:
- Don't specify a GitHub Actions runner for linuxkit builds and container image builds. - Fix image tar.gz names to be compliant with actions/upload-artifact@v4. - Make sure to export images when they are pulled from a remote registry. Signed-off-by: Jacob Weinstock <[email protected]>
1 parent c4e7632 commit 969afcb

File tree

2 files changed

+11
-6
lines changed

2 files changed

+11
-6
lines changed

.github/workflows/build-all-matrix.yaml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,10 +25,10 @@ env: # Global environment, passed to all jobs & all steps
2525
CI_TAGS: "standard armbian-sbc armbian-uefi lts" # 'dev' is not included
2626

2727
# GHA runner configuration. See bash/json-matrix.sh for more details.
28-
CI_RUNNER_LK_CONTAINERS_ARM64: "oracle-24cpu-384gb-arm64" # Use a self-hosted runner with the "ARM64" tag for the ARM64 builds of LK containers
29-
CI_RUNNER_LK_CONTAINERS_AMD64: "oracle-24cpu-384gb-x86-64" # Use a self-hosted runner with the "X86" tag for the AMD64 builds of LK containers
30-
CI_RUNNER_LK_ARM64: "oracle-24cpu-384gb-arm64" # Use a self-hosted runner with the "ARM64" tag for the ARM64 linuxkit builds
31-
CI_RUNNER_LK_AMD64: "oracle-24cpu-384gb-x86-64" # Use a self-hosted runner with the "X86" tag for the AMD64 linuxkit builds
28+
CI_RUNNER_LK_CONTAINERS_ARM64: "" # Use a self-hosted runner with the "ARM64" tag for the ARM64 builds of LK containers
29+
CI_RUNNER_LK_CONTAINERS_AMD64: "" # Use a self-hosted runner with the "X86" tag for the AMD64 builds of LK containers
30+
CI_RUNNER_LK_ARM64: "" # Use a self-hosted runner with the "ARM64" tag for the ARM64 linuxkit builds
31+
CI_RUNNER_LK_AMD64: "" # Use a self-hosted runner with the "X86" tag for the AMD64 linuxkit builds
3232
CI_RUNNER_KERNEL_AMD64: "oracle-24cpu-384gb-x86-64" # Use a self-hosted runner with the "X86" tag for the AMD64 kernel builds
3333
CI_RUNNER_KERNEL_ARM64: "oracle-24cpu-384gb-arm64" # Use a self-hosted runner with the "ARM64" tag for the ARM64 kernel builds
3434

bash/hook-lk-containers.sh

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,11 @@ function build_hook_linuxkit_container() {
5353
log debug "Checking if image ${container_oci_ref} can be pulled from remote registry"
5454
if docker pull "${container_oci_ref}"; then
5555
log info "Image ${container_oci_ref} pulled from remote registry, skipping build"
56+
# If EXPORT_LK_CONTAINERS=yes then export images as tar.gzs to EXPORT_LK_CONTAINERS_DIR
57+
# This is mainly for CI to be able to pass built images between jobs
58+
if [[ "${export_container_images}" == "yes" ]]; then
59+
save_docker_image_to_tar_gz "${container_oci_ref}" "${export_container_images_dir}"
60+
fi
5661
return 0
5762
fi
5863

@@ -89,8 +94,8 @@ function save_docker_image_to_tar_gz() {
8994
mkdir -p "${export_dir}"
9095

9196
# Save the Docker image as a tar.gz file
92-
docker save "${container_oci_ref}" | gzip > "${export_dir}/$(basename "${container_oci_ref}").tar.gz"
93-
log info "Saved Docker image ${container_oci_ref} to ${export_dir}/$(basename "${container_oci_ref}").tar.gz"
97+
docker save "${container_oci_ref}" | gzip > "${export_dir}/$(basename "${container_oci_ref}" | sed 's/:/-/g').tar.gz"
98+
log info "Saved Docker image ${container_oci_ref} to ${export_dir}/$(basename "${container_oci_ref}" | sed 's/:/-/g').tar.gz"
9499
}
95100

96101
function push_hook_linuxkit_container() {

0 commit comments

Comments
 (0)