-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WIP] CI: attach QEMU test to packages workflow #21
base: main
Are you sure you want to change the base?
Conversation
Add support for uploading compiled images if build_full is triggered. This is in preparation for testing the images in QEMU if supported. Signed-off-by: Christian Marangi <[email protected]>
@aparcar the thing |
- target: x86 | ||
subtarget: 64 | ||
use_qemu: true | ||
qemu_target: x86 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could we add armsr/armv8 here, too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
extra 40 minutes for each run but yes
- name: Run test with QEMU | ||
if: inputs.use_qemu == true | ||
run: | | ||
export LG_QEMU_BIN=$(which qemu-system-${{ inputs.qemu_target }}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I found out you can drop the which
wrapping, it was a path issue on my local device.
run: | | ||
export LG_QEMU_BIN=$(which qemu-system-${{ inputs.qemu_target }}) | ||
|
||
poetry run pytest --lg-env target_${{ inputs.target }}-${{ inputs.subtarget }}.yaml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I renamed the target to qemu.yaml
and instead added the --target
argument. Also please checkout --firmware
to specify the file, which should make things more elegant.
37b36fc
to
e4d1be9
Compare
Introduce reusable test-image workflow. Currently only QEMU is supported. This assume it's run after an artifact is uploaded by other workflow. Signed-off-by: Christian Marangi <[email protected]>
Add additional job to enable testing with QEMU using the new reusable_test-image workflow. Signed-off-by: Christian Marangi <[email protected]>
Some images are compressed with .gz format, extract them to correctly use them. Signed-off-by: Christian Marangi <[email protected]>
Attach QEMU test to packages workflow. This is still WIP as some test still needs some better handling.
The reusable workflow expect an artifact to be downloaded and tested. Currently only QEMU is supported, but it's expected that sometime in the future this is expanded also to labgrid and expanded also to other workflow... (maybe restrict this to push event?)