-
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
[YAML] Add TestMultiAdmin.yaml #15106
[YAML] Add TestMultiAdmin.yaml #15106
Conversation
For the record, I have also updated this PR so the test is not in the manual section anymore, but in the section that is automatically run by CI. |
PR #15106: Size comparison from 8a9e023 to 687f004 Increases above 0.2%:
Increases (2 builds for linux)
Full report (43 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Fun, Linux seems to have some memory issues:
|
687f004
to
cae64e3
Compare
PR #15106: Size comparison from db7ab8c to cae64e3 Increases above 0.2%:
Increases (1 build for linux)
Full report (32 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
PR #15106: Size comparison from db7ab8c to 820a138 Increases above 0.2%:
Increases (4 builds for linux, nrfconnect)
Full report (32 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
There is still a leak in minimal. Short summary is that the connectedhomeip/src/lib/support/Pool.h Line 217 in 01ae39f
There is still something unclear to me as I don't get why |
/rebase |
820a138
to
614e713
Compare
PR #15106: Size comparison from e03e5d3 to 614e713 Increases above 0.2%:
Increases (3 builds for linux, nrfconnect)
Full report (30 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
/rebase |
614e713
to
68b5234
Compare
PR #15106: Size comparison from dbbc6e9 to 68b5234 Increases above 0.2%:
Increases (2 builds for linux)
Full report (31 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
/rebase |
68b5234
to
5bd7c7d
Compare
PR #15106: Size comparison from d6bf695 to 5bd7c7d Increases above 0.2%:
Increases (2 builds for linux)
Full report (31 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
/rebase |
5bd7c7d
to
2ceb27f
Compare
PR #15106: Size comparison from dde4a8f to 2ceb27f Increases above 0.2%:
Increases (2 builds for linux)
Full report (31 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
* Add TestMultiAdmin * Update generated code
Problem
This PR adds a test that makes uses of multiple controllers/fabrics to control a device.
I was hoping to add it to CI directly, but since it seems that there is still some flakiness on Darwin with the mdns advertisement I am just adding to it the manual section at the moment. So that may show to others how to use multiple fabrics from a single test.
Change overview
TestMultiAdmin.yaml
Testing
The test has been ran manually multiple times.