ci: add cv200_neo + av100_neo qemu-boot matrix rows#181
Merged
Conversation
…m firmware) Now that the firmware nightly publishes openipc.hi3516cv200-nor-neo.tgz and openipc.hi3516av100-nor-neo.tgz (after OpenIPC/firmware#2124 added both boards to the firmware build.yml matrix), wire two new rows into the qemu-boot job. Each row downloads the appropriate -nor-neo.tgz from the latest firmware release and runs the same boot smoke as the lite rows. Local pre-merge validation against the published nightly tarballs: hi3516cv200_neo: [PASS] login (openipc-hi3516cv200 login:) [PASS] IP (10.0.2.15 via SLIRP DHCP) [PASS] ping 10.0.2.2 hi3516av100_neo: [PASS] login (openipc-hi3516av100 login:) [PASS] IP (10.0.2.15 via SLIRP DHCP) [PASS] ping 10.0.2.2 Both also clean on Oops/panic/BUG/Call Trace patterns. Row mechanics — new optional `variant:` field: - Defaults to lite. Rows that don't set it stay byte-equivalent (download URL becomes openipc.X-nor-lite.tgz, row name unchanged). - When `variant: neo`: * row name becomes "QEMU boot (<machine>_neo)" * download URL becomes openipc.<machine>-nor-neo.tgz * upload-artifact name becomes "qemu-boot-<machine>_neo" Both new rows are real gates (no allow-failure: true). load_hisilicon on neo doesn't load HiSi blobs (kernel-version path mismatch — separate firmware follow-up to teach it the /lib/modules/7.0 path), so neither row sets min_modules; the assertion is "login reached + no error patterns". That's what the local DoD test exercises and what the CI will exercise. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.
Summary
Closes OpenIPC/firmware#2123. Now that the firmware nightly publishes
openipc.hi3516cv200-nor-neo.tgzandopenipc.hi3516av100-nor-neo.tgz(after OpenIPC/firmware#2124 added both boards to the firmware build matrix), wire two new rows into the qemu-boot job.Local pre-merge validation
Both nightly tarballs were downloaded and exercised against the same smoke harness this PR's CI will use:
Row mechanics
New optional
variant:field on a row:lite. Rows that don't set it stay byte-equivalent (download URLopenipc.X-nor-lite.tgz, row name unchanged).variant: neo:QEMU boot (<machine>_neo)openipc.<machine>-nor-neo.tgzqemu-boot-<machine>_neoReal gate, not allow-failure
Both new rows are real gates — no
allow-failure: true.load_hisiliconon neo doesn't load HiSi blobs (kernel-version path mismatch — separate firmware follow-up to teach it the/lib/modules/7.0path), so neither row setsmin_modules:; the assertion is "login reached + no error patterns". That's what the local DoD test exercises and what the CI will exercise.Test plan
SUCCESSbefore merge (branch protection enforces all 27 required checks)🤖 Generated with Claude Code