-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Add initial Arm Virtual Hardware (AVH) workflow with end to end test #30250
Conversation
PR #30250: Size comparison from 6b60c62 to 5283105 Full report (73 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
it seems test is still failing, is this caused by permission in github setting?
|
@yunhanw-google I believe this is because the |
PR #30250: Size comparison from 5c4a9c8 to 685b444 Full report (73 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
@andy31415 thank you for taking the time to review and your valuable feedback. I've pushed some commits to incorporate your feedback. There's one open item on #30250 (comment) about the cancel in progress. Let me know if you have ideas on how to move forward with it. |
@sandeepmistry restyler says some changes are needed. I believe this is why we do not have reviewers set yet. |
PR #30250: Size comparison from 53c1608 to 3c60648 Full report (56 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg)
|
PR #30250: Size comparison from 53c1608 to 710e19a Increases above 0.2%:
Increases (73 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
Decreases (5 builds for linux, psoc6)
Full report (73 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
Old PR with failing CI. Assuming stale and closing. |
This pull request adds a new Arm Virtual Hardware (AVH) based GitHub Action flow that performs an end-to-end test of the
chip-tool
andlighting-app
binaries using two virtual Raspberry Pi 4 boards. It is an updated version of #25686 which uses the new csa.app.avh.arm.com AVH instance and isolated AVH projects per test run.The workflow consists of two parts:
Cross-compiling the
chip-tool
andlighting-app
binaries for ArmThe end-to-end test which
a) Creates a new AVH project with an isolated virtual Wi-Fi and BLE network using the AVH API
b) Starts up two virtual Raspberry Pi 4 instances using the AVH API
c) Configuration of the two virtual Raspberry Pi 4 instances: one as a lighting device and one as a controller
d) Commissioning of the lighting device using BLE with Wi-Fi credentials
e) Testing the light on and off command
f) Deletes the two virtual Raspberry Pi 4 instances
g) Deletes the AVH project
The GitHub Action settings for the repo with need to have the
AVH_API_TOKEN
environment variable set which I can provide a value for.cc/ @sondhiArm