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 <ansuelsmth@gmail.com>
Member
Author
|
@aparcar the thing |
aparcar
requested changes
May 6, 2024
| - target: x86 | ||
| subtarget: 64 | ||
| use_qemu: true | ||
| qemu_target: x86 |
Member
There was a problem hiding this comment.
could we add armsr/armv8 here, too?
Member
Author
There was a problem hiding this comment.
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 }}) |
Member
There was a problem hiding this comment.
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 |
Member
There was a problem hiding this comment.
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.
Ansuel
force-pushed
the
dev
branch
6 times, most recently
from
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 <ansuelsmth@gmail.com>
Add additional job to enable testing with QEMU using the new reusable_test-image workflow. Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
Some images are compressed with .gz format, extract them to correctly use them. Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
Ansuel
force-pushed
the
main
branch
10 times, most recently
from
7ef03b6 to
0a3f3c9
Compare
Ansuel
force-pushed
the
main
branch
2 times, most recently
from
94e64ba to
c7ec823
Compare
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
2 participants
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.
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?)