Skip to content
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

[BUG] [TC-CADMIN-1.9] ECM commissioning retry counter is incrementing faster than expected #24971

Closed
adamb-q opened this issue Feb 10, 2023 · 4 comments · Fixed by #25176
Closed

Comments

@adamb-q
Copy link
Contributor

adamb-q commented Feb 10, 2023

Reproduction steps

Platform: qpg, chip-tool x64

Running CADMIN 1.9 shows that using ECM commissioning counter is incrementing faster than expected, which causes test failure earlier than on 20th retry. When using BCM the issue is not happening - 1 failure per 1 attempt. Attaching logs from CADMIN 1.9 and 1.10 for reference.

Bug prevalence

Every time

GitHub hash of the SDK that was being used

7e69c66

Platform

other

Platform Version(s)

qpg

Anything else?

TC_CADMIN_1_9.1675432562.log
TC_CADMIN_1_9.1675432562_DUT.log
[TC-CADMIN-1.10]_[061nwOu0]_CHIPTool_log.log
TC_CADMIN_1_10.1675433327_DUT.log

@adamb-q adamb-q changed the title [BUG] [TC-CADMIN-1.9] Commissioning retry counter is incrementing faster than expected [BUG] [TC-CADMIN-1.9] ECM commissioning retry counter is incrementing faster than expected Feb 10, 2023
@bzbarsky-apple
Copy link
Contributor

@adamb-q Is this a duplicate of #24928 ?

@adamb-q
Copy link
Contributor Author

adamb-q commented Feb 13, 2023

@bzbarsky-apple I applied the --discover-once true flag, and it looks better, but still the log output from chip-tool gives the correct output only on 1st attempt, I mean Failed to verify peer's MAC. This can happen when setup code is incorrect. The next steps differ, can I assume that it is correct behavior also? Attaching logs with the discover-once flag set.

TC_CADMIN_1_9.1676278577_DUT.log
TC_CADMIN_1_9.1676278577.log

@bzbarsky-apple
Copy link
Contributor

That sounds like #22547 ?

@adamb-q
Copy link
Contributor Author

adamb-q commented Feb 14, 2023

Yes, looks relevant. When waiting ~50s between retries test would pass, but it exceeds 900s and commissioning window closes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants