-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Document how to add new devices via CHEF #27900
Conversation
PR #27900: Size comparison from 77b8d9f to 8851e31 Increases (42 builds for bl602, bl702, bl702l, cc32xx, efr32, esp32, linux, psoc6, telink)
Decreases (26 builds for cyw30739, efr32, esp32, k32w, linux, nrfconnect, qpg)
Full report (58 builds for bl602, bl702, bl702l, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
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.
How will it scale in terms of CI vs all-clusters-apps? We will likely have a ton of device-type chef apps to build and then test.
Platforms might want to run the ci with their platform also, not just Linux.
Co-authored-by: Erwin Pan <erwinpan1@gmail.com>
Co-authored-by: Erwin Pan <erwinpan1@gmail.com>
Co-authored-by: Erwin Pan <erwinpan1@gmail.com>
Co-authored-by: Erwin Pan <erwinpan1@gmail.com>
Co-authored-by: Erwin Pan <erwinpan1@gmail.com>
Co-authored-by: Erwin Pan <erwinpan1@gmail.com>
@andy31415 , this PR of yours is breaking the documentation structure consistency. Kindly please fix this as per issue #28354 |
We need to somehow scale adding new device types and validating them (mostly from a controller perspective) and all-clusters-app seems to have scaling issues.
Would like to have the minimal path required to get fake device up and running, for basic controller tests (i.e. does this device show up), chef works.
Will have to figure out future guidance for developers:
For now documenting the shortest path of "get a virtual device of type X running"