-
Notifications
You must be signed in to change notification settings - Fork 7
QNX8 QEMU x86_64 Integration tests #19
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
base: main
Are you sure you want to change the base?
Changes from 36 commits
40329f1
fdcc905
f8de33b
0df5e9b
5affffc
b35d951
128cfb8
4e75fd4
bb70f92
ebbdd65
db971ff
2915a07
05f317e
ce061e2
c12b49f
4ef6e72
a920ffe
92e4730
f815597
8fc1bfe
897f38b
73dc8e0
08ff0e4
8f4447f
a8b5bb1
0ffb175
6613fad
2cc5f03
e377469
4589bac
4a6f50e
8bc16d4
905fca8
2b8f18f
75f77df
83e3d3b
8dd5f55
d5ec295
4b31637
de9dedc
05ab324
d7a70fb
fa911e5
8381329
fc18cc1
44d60ec
61a08de
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -4,7 +4,9 @@ | |||||
| "dockerfile": "Dockerfile" | ||||||
| }, | ||||||
| "runArgs": [ | ||||||
| "--privileged" | ||||||
| "--privileged", | ||||||
| "--add-host=host.docker.internal:host-gateway", | ||||||
| "--device=/dev/kvm" | ||||||
| ], | ||||||
| "mounts": [ | ||||||
| { | ||||||
|
|
@@ -31,5 +33,7 @@ | |||||
| "streetsidesoftware.code-spell-checker" | ||||||
| ] | ||||||
| } | ||||||
| } | ||||||
| }, | ||||||
| "postCreateCommand": "sudo apt-get update && sudo apt-get install -y software-properties-common && sudo add-apt-repository -y universe && sudo apt-get update && sudo apt-get install -y qemu-system-x86 iputils-ping tcpdump iptables", | ||||||
|
||||||
| "postCreateCommand": "sudo apt-get update && sudo apt-get install -y software-properties-common && sudo add-apt-repository -y universe && sudo apt-get update && sudo apt-get install -y qemu-system-x86 iputils-ping tcpdump iptables", | |
| "postCreateCommand": "sudo apt-get update && sudo apt-get install -y --no-install-recommends software-properties-common && sudo add-apt-repository -y universe && sudo apt-get update && sudo apt-get install -y --no-install-recommends qemu-system-x86 iputils-ping tcpdump iptables", |
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.
updated
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -1,4 +1,3 @@ | ||||||
|
|
||||||
| # SOME/IP Gateway | ||||||
|
|
||||||
| The gateway is divided into a gateway daemon (gatewayd) which contains the network-independent logic (payload serialization, etc.) and the SOME/IP daemon (someipd) which binds to the concrete SOME/IP stack. | ||||||
|
|
@@ -41,42 +40,37 @@ bazel run //examples/car_window_sim:car_window_controller | |||||
| If you type `open` or `close` the command will be sent via network. | ||||||
|
|
||||||
|
|
||||||
| ### Dockerized integration test POC | ||||||
| ### QEMU x86_64 - based integration test and unit tests | ||||||
|
|
||||||
| For integration tests where the communication between two QEMU instances is required, a custom implementation is used to start and manage the QEMU instances within the test logic. This is because ITF does not support starting multiple QEMU instances in parallel yet. | ||||||
|
|
||||||
| For integration tests, a docker based approach was taken. | ||||||
| As a proof of concept `docker compose` can be used to build, setup and run the containers. | ||||||
| In the future a pytest based setup can be implemented to orchestrate the containers. | ||||||
| For the QEMU QNX x864 image to run on host please run the script deployment/qemu/setup_bridge.sh with sudo privileges to setup the required network bridge and tap interfaces. | ||||||
| It is stronly recommended to run all tests with `--nocache_test_results` which is the best way on development cycles to ensure you are always running the latest version of the tests and not accidentally running cached results. | ||||||
|
||||||
| It is stronly recommended to run all tests with `--nocache_test_results` which is the best way on development cycles to ensure you are always running the latest version of the tests and not accidentally running cached results. | |
| It is recommended to run all tests with `--nocache_test_results` to ensure you are always running the latest version of the tests and not accidentally seeing cached results. |
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.
updated
Outdated
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.
| Tu run all tests (will take around 2 minutes) | |
| To run all tests (will take around 2 minutes): |
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.
updated
Outdated
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.
This is only needed in devcontainer or also when running locally?
And why don't we add it to the devcontainer file so that it gets executed automatically? Is there a negative impact?
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.
First question : is needed independent if its host or docker containers .
Second question:
Done
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.
Please add that strategy to those tests which need to be executed locally. At the very least it should be limited to the
testcommand and not incommonUh oh!
There was an error while loading. Please reload this page.
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.
updated to test only form qnx arm anx x86_64