Skip to content

perf(ci): built the unit test image in the debug build jobs instead of the unit-test jobs - #344

Merged
FlareCoding merged 2 commits into
masterfrom
pr/ci-test-image-artifact
Oct 6, 2026
Merged

FlareCoding merged 2 commits into
masterfrom
pr/ci-test-image-artifact

Conversation

@FlareCoding

@FlareCoding FlareCoding commented Oct 6, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Each unit-test job rebuilt the whole test image, and about three of its five minutes went to compiling a userland the build jobs had already compiled.
  • make test-image is the build half of make test. The debug build jobs, which build in make test's default mode, run it on top of the userland they just compiled and upload the image, and the unit-test jobs boot that image with only QEMU installed.
  • The image comes from the same run through the same Makefile path, so every commit still exercises the full image build, and a run drops from about 9.5 to 7 minutes.

Note

Low Risk
CI/Makefile orchestration only; local make test behavior is preserved via the test-image dependency with no kernel or test logic changes.

Overview
CI now builds the QEMU unit-test disk image in debug build jobs and reuses it in unit-tests, instead of rebuilding userland twice per pipeline run.

The Makefile splits make test into make test-image (clean + unit-test-enabled image build) and make test, which still runs test-image then scripts/run_tests.sh. Debug matrix entries run make test-image with PACKAGES="none binutils" after kernel/userland and upload images/stellux-<arch>.img as a short-lived artifact.

The unit-tests job drops toolchain/sysroot caches and Limine/musl/libc++ setup; it only installs QEMU, downloads the prebuilt image, and invokes ./scripts/run_tests.sh directly (same timeout and log upload as before).

Reviewed by Cursor Bugbot for commit 274a18c. Bugbot is set up for automated code reviews on this repo. Configure here.

FlareCoding and others added 2 commits October 5, 2026 23:52
Co-authored-by: Cursor <cursoragent@cursor.com>
…f the unit-test jobs

Co-authored-by: Cursor <cursoragent@cursor.com>
@FlareCoding
FlareCoding merged commit 095e194 into master Oct 6, 2026
19 checks passed
@FlareCoding
FlareCoding deleted the pr/ci-test-image-artifact branch October 6, 2026 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant