-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Ble-WiFi commissioning test in CI #32889
base: master
Are you sure you want to change the base?
Conversation
Right now, I'm using a docker image from my fork. After merging the changes required to create this image in the main repository, I will change the address and make this PR ready for merge. I created this PR to show a possible solution and get feedback. |
PR #32889: Size comparison from f46cdf3 to 4c71adb Decreases (1 build for efr32)
Full report (71 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink)
|
3848f88
to
7e54edb
Compare
PR #32889: Size comparison from f29ccbe to 7e54edb Decreases (1 build for efr32)
Full report (71 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink)
|
PR #32889: Size comparison from f29ccbe to 03a8c1e Decreases (1 build for efr32)
Full report (71 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink)
|
"pairing", "ble-wifi", TEST_NODE_ID, "Virtual_Wifi", "ExamplePassword", "20202021", "3840", ] | ||
pairing_server_args = [ | ||
"--ble-adapter", str(tool_hci_number)] |
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.
pairing code-wifi
should work, no? Why is this using ble-wifi
, which is not really maintained?
PR #32889: Size comparison from 0180ee6 to a46b697 Full report (81 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink)
|
PR #32889: Size comparison from 3219a5f to 88a573c Decreases (2 builds for efr32)
Full report (81 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink)
|
PR #32889: Size comparison from 4f0b7e9 to 1310792 Increases above 0.2%:
Full report (81 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, psoc6, qpg, stm32, telink, tizen)
|
Problem
There is no testing of commissioning with ble-wifi in the CI
Solution
Currently, there is no way to use virtual Bluetooth and wifi interfaces in CI. Github runner doesn't have
required modules to create virtual interfaces: hci_vhci and mac80211_hwsim.
The proposed solution involves creating a custom qemu image with the required modules and using it in CI.
The created qemu image is based on ubuntu:20.04 with a custom kernel and Bluez installed. The patch used to make the custom kernel is required to ensure that two devices can discover each other every time.
Tests use their dbus-daemon instance to separate them from the system dbus. This will allow developers
to run tests on their main system without needing a virtual machine. Unfortunately, there is no way to run two instances of Bluez simultaneously and select hci interfaces for each. That's why there is a proxy between the system and test buses, which forwards Bluez messages between them.
Changes