Separate image build to each workflow#352
Merged
yockgen merged 9 commits intoopen-edge-platform:mainfrom Jan 20, 2026
Merged
Conversation
Signed-off-by: Teoh Suh Haw <suh.haw.teoh@intel.com>
Signed-off-by: Teoh Suh Haw <suh.haw.teoh@intel.com>
Signed-off-by: Teoh Suh Haw <suh.haw.teoh@intel.com>
Signed-off-by: Teoh Suh Haw <suh.haw.teoh@intel.com>
Signed-off-by: Teoh Suh Haw <suh.haw.teoh@intel.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR separates a monolithic validation script into individual build scripts for each image type and creates dedicated GitHub Actions workflows to build images independently. Previously, all image builds ran sequentially in a single script; now each image type (raw, ISO, immutable) for each distribution (AZL3, EMT3, ELXR12, Ubuntu24) has its own script and can be triggered via dedicated workflows.
Changes:
- Removed validate.sh containing all sequential builds
- Created 12 individual build scripts in scripts/ directory (one per image type/distribution combination)
- Added 12 new GitHub Actions workflow files for individual image builds
- Updated boot_tester.yml to use matrix strategy with individual build scripts
- Renamed and updated build-azl3-raw.yml workflow
Reviewed changes
Copilot reviewed 26 out of 26 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| validate.sh | Removed monolithic validation script containing all builds |
| scripts/build_ubuntu24_raw.sh | New script for Ubuntu 24 raw image build |
| scripts/build_ubuntu24_iso.sh | New script for Ubuntu 24 ISO image build |
| scripts/build_ubuntu24_immutable.sh | New script for Ubuntu 24 immutable image build |
| scripts/build_emt3_raw.sh | New script for EMT3 raw image build |
| scripts/build_emt3_iso.sh | New script for EMT3 ISO image build |
| scripts/build_emt3_immutable.sh | New script for EMT3 immutable image build |
| scripts/build_elxr12_raw.sh | New script for ELXR12 raw image build |
| scripts/build_elxr12_iso.sh | New script for ELXR12 ISO image build |
| scripts/build_elxr12_immutable.sh | New script for ELXR12 immutable image build |
| scripts/build_azl3_raw.sh | New script for AZL3 raw image build |
| scripts/build_azl3_iso.sh | New script for AZL3 ISO image build |
| scripts/build_azl3_immutable.sh | New script for AZL3 immutable image build |
| .github/workflows/build-ubuntu24-raw.yml | New workflow for Ubuntu 24 raw image build |
| .github/workflows/build-ubuntu24-iso.yml | New workflow for Ubuntu 24 ISO image build |
| .github/workflows/build-ubuntu24-immutable.yml | New workflow for Ubuntu 24 immutable image build |
| .github/workflows/build-emt3-raw.yml | New workflow for EMT3 raw image build |
| .github/workflows/build-emt3-iso.yml | New workflow for EMT3 ISO image build |
| .github/workflows/build-emt3-immutable.yml | New workflow for EMT3 immutable image build |
| .github/workflows/build-elxr12-raw.yml | New workflow for ELXR12 raw image build |
| .github/workflows/build-elxr12-iso.yml | New workflow for ELXR12 ISO image build |
| .github/workflows/build-elxr12-immutable.yml | New workflow for ELXR12 immutable image build |
| .github/workflows/build-azl3-raw.yml | Updated existing workflow to use new AZL3 raw build script |
| .github/workflows/build-azl3-iso.yml | New workflow for AZL3 ISO image build |
| .github/workflows/build-azl3-immutable.yml | New workflow for AZL3 immutable image build |
| .github/workflows/boot_tester.yml | Updated to use matrix strategy with individual build scripts |
Comments suppressed due to low confidence (4)
scripts/build_ubuntu24_iso.sh:1
- The variable RAW_IMAGE is set in the bash -c block but is never defined in the run_qemu_boot_test_iso function, making this assignment set it to an empty value. This appears to be copied from the raw image test function but is unnecessary for ISO testing.
#!/bin/bash
scripts/build_emt3_iso.sh:1
- The variable RAW_IMAGE is set in the bash -c block but is never defined in the run_qemu_boot_test_iso function, making this assignment set it to an empty value. This appears to be copied from the raw image test function but is unnecessary for ISO testing.
#!/bin/bash
scripts/build_elxr12_iso.sh:1
- The variable RAW_IMAGE is set in the bash -c block but is never defined in the run_qemu_boot_test_iso function, making this assignment set it to an empty value. This appears to be copied from the raw image test function but is unnecessary for ISO testing.
#!/bin/bash
scripts/build_azl3_iso.sh:1
- The variable RAW_IMAGE is set in the bash -c block but is never defined in the run_qemu_boot_test_iso function, making this assignment set it to an empty value. This appears to be copied from the raw image test function but is unnecessary for ISO testing.
#!/bin/bash
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Teoh Suh Haw <suh.haw.teoh@intel.com>
274a529 to
b008dd6
Compare
yockgen
approved these changes
Jan 20, 2026
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.
Merge Checklist
All boxes should be checked before merging the PR
Description
Any Newly Introduced Dependencies
How Has This Been Tested?