-
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
TC-IDM-10.1 - add duplicate value check #28892
TC-IDM-10.1 - add duplicate value check #28892
Conversation
PR #28892: Size comparison from 8e806c5 to db4fb6f Increases (1 build for telink)
Decreases (2 builds for efr32, telink)
Full report (62 builds for bl602, bl702, bl702l, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
PR #28892: Size comparison from 79292ad to 3942afa Decreases (1 build for efr32)
Full report (62 builds for bl602, bl702, bl702l, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
PR #28892: Size comparison from 79292ad to 1e0f96c Increases (19 builds for bl702, bl702l, cc32xx, cyw30739, esp32, linux, nrfconnect, psoc6, telink)
Decreases (1 build for efr32)
Full report (62 builds for bl602, bl702, bl702l, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
We seem to get:
|
Guessing a merge with master is needed? but then we need to fix file conflicts. |
…for_global_attributes
PR #28892: Size comparison from f8690ac to 5e06a22 Decreases (1 build for efr32)
Full report (67 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
Error changed to
However I somehow cannot find what actual element is missing (the log is quite spammy for all clusters as there are a lot of clusters there and all of them say "found" |
Hard to find because the error is duplication, it says "general diagnostics found" two times. I believe we should keep track of errors separately and print them out together with the final error (instead of success=..." we should have |
This should not have happened, will have to investigate why (I thought we fixed this) |
endpoint_config still contains:
|
Ummm... it turns out #28890 fixed the TEST input not the actual app input :( |
PR #28892: Size comparison from 354120d to f8677d2 Decreases (1 build for efr32)
Full report (67 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
* TC-IDM-10.1 - add duplicate value check * simplify checker call
* TC-IDM-10.1 - add duplicate value check * simplify checker call
This is going to fail CI until andrei's change goes in because we're currently failing this test on all clusters
Test: Current all-clusters fails this test