Skip to content

CI smoketest on a virtualized A55 CPU - #33

Merged
mcm001 merged 6 commits into
mainfrom
ci-test-instructions
Aug 1, 2026
Merged

CI smoketest on a virtualized A55 CPU#33
mcm001 merged 6 commits into
mainfrom
ci-test-instructions

Conversation

@mcm001

@mcm001 mcm001 commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

In https://github.com/PhotonVision/mrcal-java/pull/27/changes we added some flags to make sure our Arm64 builds targeted armv8-a, because the Github Actions runner we build on's native architecture includes instructions illegal to run on Orange Pi 5/Pi devices. Let's add a CI test to make sure that we can run on a simulated cortex A55 CPU.

This adds 27 seconds of runtime -- 21 to run the test under QEMU, and 6 to install packages. Windows is still critical path at 10.5 minutes, though

image

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates CI to execute an ARM64 smoketest of mrcal_jni_test under qemu-aarch64 (Cortex-A55), and reduces the runtime of the native test executable to better fit CI constraints.

Changes:

  • Reduce the mrcal_jni_test loop iteration count (5 → 1) to shorten execution time.
  • Extend the build workflow to run mrcal_jni_test and to stage/upload ARM64 binaries and shared-library artifacts.
  • Add a follow-on job that downloads the ARM64 artifacts and runs the test under QEMU with -cpu cortex-a55.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/mrcal_test.cpp Reduces test/benchmark iteration count to speed up CI smoketests.
.github/workflows/main.yml Builds/runs native test, stages ARM64 runtime artifacts, and adds a QEMU A55 execution job.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/main.yml
@mcm001
mcm001 merged commit 319f569 into main Aug 1, 2026
6 checks passed
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.

2 participants