-
Notifications
You must be signed in to change notification settings - Fork 13
feat: Use more generic ITF QEMU plugin #177
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
Draft
lurtz
wants to merge
62
commits into
eclipse-score:main
Choose a base branch
from
elektrobit-contrib:make-qemu-linux-plugin-more-generic
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from 59 commits
Commits
Show all changes
62 commits
Select commit
Hold shift + click to select a range
8840793
apply patch with adaptions from ITF to run EB Linux
lurtz 3898a5d
Add network device to architecture dependent parameters
lurtz a2516b4
Somehow use EB Linux in test
lurtz f181495
directly use py_itf_test for running eb corbos linux image
lurtz dfd95c7
use same itf hash as declared with version
lurtz 8ad6cc4
remove unused file
lurtz ff31795
single location for all ebclfsa targets
lurtz 18e270d
put most reference_integration dependencies into extra file
lurtz 11c5def
Merge remote-tracking branch 'origin/main' into make-qemu-linux-plugi…
lurtz ab9f5be
unify qemu image type determination
lurtz 2d53560
move EB Linux specific settings out of the plugin
lurtz 805165d
remove non essential qemu parameters
lurtz 70c2f56
more simplification
lurtz 9d2fffb
kind of QNX support for linux_qemu plugin
lurtz b328ac2
move architecture dependent QEMU parameters to map
lurtz 6f7032e
reintroduce some QEMU parameters with comments for reason
lurtz 63346de
make kernel commandline optional
lurtz 94a2162
fix running QNX via linux_qemu plugin
lurtz ee10669
simplify QEMU acceleration
lurtz 2efed81
unify machine and kernel_machine
lurtz f353628
always add machine flag
lurtz 91d0065
simplify qemu command line creation
lurtz 8c61a75
make image parameter optional
lurtz 60a673e
remove qemu boot mode
lurtz 21367fc
simplify disk image code
lurtz 6629033
Align seed drive logic with that of the image
lurtz 1937361
remove unused members and functions
lurtz e0879dd
remove reference_integration dependency
lurtz 0ace1ee
remove leftovers
lurtz c618f12
minimize lock file changes
lurtz 1404ed9
install aarch64 qemu for integration tests
lurtz e0ac51f
only build Ebclfsa image when explicitly needed
lurtz 663f0ce
Only build EB Linux image when requested
lurtz ada9ab0
Merge remote-tracking branch 'origin/main' into make-qemu-linux-plugi…
lurtz ed74c1b
simplify and speedup SSH retry logic
lurtz f91040e
Merge branch 'main' into make-qemu-linux-plugin-more-generic
lurtz 9c8cb48
Use improved itf qemu plugin (#4)
lurtz 465c8b7
remove unused file
lurtz 3996a68
Merge remote-tracking branch 'origin/main' into make-qemu-linux-plugi…
lurtz 2980db0
remove wasting image copy
lurtz f76a26a
install guestfish for Linux integration tests
lurtz fa1eb64
tag rootfs as manual
lurtz a6f1d60
tag ebclfsa as manual
lurtz 7b161ad
attempt to fix building the images
lurtz 7100db3
ci: allow unprivileged user namespaces for qemu integration
lurtz ebb85f0
ci: run CopyFilesOntoImage outside sandbox in qemu job
lurtz 38e242d
ci: pin supermin kernel and libguestfs runtime dirs
lurtz 8baa979
ci: pass libguestfs env via bazel action_env
lurtz 4dc2fdf
ci: use fixed libguestfs appliance for overlay actions
lurtz 1f22998
ci: build fixed appliance with LIBGUESTFS_PATH unset
lurtz 6fd8402
ci: make runner kernel readable for supermin
lurtz 9060d12
ci: increase qemu pre-test ping timeout in score_itf
lurtz effe0a2
use most recent branch revision
lurtz 3c6253f
small workflow simplification
lurtz ce282b3
more workflow simplification
lurtz 85f451a
remove make-fixed-appliance
lurtz 22e700f
remove strategy for CopyFilesOntoImage
lurtz e66d60d
make job abort if image cannot be made readable
lurtz e299b49
use build_image.sh from ITF
lurtz 1d3d8ae
explicitly enable root
lurtz ad1c5fd
bump ITF again
lurtz 9690b61
Merge branch 'main' into make-qemu-linux-plugin-more-generic
lurtz File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
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
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
41 changes: 41 additions & 0 deletions
41
quality/integration_testing/environments/ebclfsa_aarch64_qemu/BUILD
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,41 @@ | ||
| # ******************************************************************************* | ||
| # Copyright (c) 2026 Contributors to the Eclipse Foundation | ||
| # | ||
| # See the NOTICE file(s) distributed with this work for additional | ||
| # information regarding copyright ownership. | ||
| # | ||
| # This program and the accompanying materials are made available under the | ||
| # terms of the Apache License Version 2.0 which is available at | ||
| # https://www.apache.org/licenses/LICENSE-2.0 | ||
| # | ||
| # SPDX-License-Identifier: Apache-2.0 | ||
| # ******************************************************************************* | ||
|
|
||
| exports_files( | ||
| [ | ||
| "qemu_config.json", | ||
| ], | ||
| visibility = ["//:__subpackages__"], | ||
| ) | ||
|
|
||
| filegroup( | ||
| name = "qemu_config", | ||
| srcs = ["qemu_config.json"], | ||
| visibility = ["//:__subpackages__"], | ||
| ) | ||
|
|
||
| # Convenience alias for accessing the image | ||
| alias( | ||
| name = "image", | ||
| actual = "//quality/integration_testing/images/ebclfsa_aarch64:image", | ||
| tags = ["manual"], | ||
| visibility = ["//visibility:public"], | ||
| ) | ||
|
|
||
| # Convenience alias for accessing the kernel image | ||
| alias( | ||
| name = "kernel", | ||
| actual = "//quality/integration_testing/images/ebclfsa_aarch64:kernel", | ||
| tags = ["manual"], | ||
| visibility = ["//visibility:public"], | ||
| ) |
14 changes: 14 additions & 0 deletions
14
quality/integration_testing/environments/ebclfsa_aarch64_qemu/qemu_config.json
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,14 @@ | ||
| { | ||
| "networks": [ | ||
| { | ||
| "name": "tap0", | ||
| "ip_address": "169.254.158.190", | ||
| "gateway": "169.254.21.88" | ||
| } | ||
| ], | ||
| "ssh_port": 22, | ||
| "qemu_num_cores": 2, | ||
| "qemu_ram_size": "1G", | ||
| "qemu_machine": "virt-aarch64", | ||
| "qemu_kernel_cmdline": "root=/dev/vda1 sdk_enable lisa_syscall_whitelist=2026 rw sharedmem.enable_sharedmem=0 init=/usr/bin/ebclfsa-cflinit" | ||
| } |
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
83 changes: 0 additions & 83 deletions
83
quality/integration_testing/environments/ubuntu24_04_qemu/prepare_image.sh
This file was deleted.
Oops, something went wrong.
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -9,5 +9,5 @@ | |
| "ssh_port": 22, | ||
| "qemu_num_cores": 2, | ||
| "qemu_ram_size": "2G", | ||
| "boot_mode": "disk" | ||
| "qemu_machine": "pc-x86_64" | ||
| } | ||
Oops, something went wrong.
Oops, something went wrong.
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.
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.
Still unsure if the chosen method to copy files into images will stay due to kernel requirement.