-
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
Fix EFR32 unit test #11073
Fix EFR32 unit test #11073
Conversation
PR #11073: Size comparison from f1b8e2c to 1ae543a Full report (8 builds for k32w, p6, qpg, telink)
|
1ae543a
to
953cea9
Compare
PR #11073: Size comparison from f1b8e2c to 953cea9 Increases (1 build for esp32)
Full report (37 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
@rgoliver I see more and more test disabled on EFR32 is it more than the HEAP limit that you hit before with a crypto test? What kind of issues are preventing the EFR32 platfrom from running those? I will be available tomorrow get my hand on this test driver if you can point me to the issues you see. |
bd45a9d
to
06c75ff
Compare
PR #11073: Size comparison from 3b4fcf0 to 06c75ff Full report (37 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
06c75ff
to
8718767
Compare
PR #11073: Size comparison from 1073bb2 to 8718767 Full report (37 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
8718767
to
68cb38c
Compare
(#10447): App test has HF on EFR32.Lines 76 to 80 in 68cb38c
This comment was generated by todo based on a
|
(#10447): Test has HF on EFR32.connectedhomeip/src/messaging/tests/BUILD.gn Lines 51 to 57 in 68cb38c
This comment was generated by todo based on a
|
- Add to build script so it gets compiled in CI - Remove broken tests so it passes (174 Tests) - Fix runner to stream output - Bundle in runner whls into output - Add to smoke-test
68cb38c
to
75110e3
Compare
PR #11073: Size comparison from 8716608 to 75110e3 Decreases (1 build for esp32)
Full report (37 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Review comments addressed
- Add to build script so it gets compiled in CI - Remove broken tests so it passes (174 Tests) - Fix runner to stream output - Bundle in runner whls into output - Add to smoke-test
Problem
The EFR32 Unit test has been broken for almost 2 weeks: #10447
Change overview
Testing
/scripts/build/build_examples.py --target-glob '*-brd4161a-{lock,light,unit-test}*' build